[PHOENIX-6287] Fix incorrect log in ParallelIterators.submitWork

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 5.1.0, 4.16.0
• Fix Version/s: 5.1.0, 4.16.0
• Assignee: chenglei
• Reporter: chenglei
• Created: 30/Dec/20 09:28
• Updated: 15/Jan/21 12:38
• Resolved: 30/Dec/20 11:11

Description

In ParallelIterators.submitWork, the following log is critical but is meaningless:

123                @Override
124                public PeekingResultIterator call() throws Exception {
125                    long startTime = EnvironmentEdgeManager.currentTimeMillis();
126                    if (LOGGER.isDebugEnabled()) {
127                        LOGGER.debug(LogUtil.addCustomAnnotations("Id: " + scanId + ", Time: " +
128                            (EnvironmentEdgeManager.currentTimeMillis() - startTime) +
129                            "ms, Scan: " + scan, ScanUtil.getCustomAnnotations(scan)));
130                    }

The log should be placed after the iterator.peek is called.