vvysotskyi commented on a change in pull request #2026: DRILL-7330: Implement metadata usage for all format plugins URL: https://github.com/apache/drill/pull/2026#discussion_r395148096
########## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/ScanBatch.java ########## @@ -242,12 +242,12 @@ private IterOutcome internalNext() { if (implicitValues != null) { String projectMetadataColumn = context.getOptions().getOption(ExecConstants.IMPLICIT_PROJECT_METADATA_COLUMN_LABEL_VALIDATOR); if (recordCount > 0) { - // Sets the implicit value to false to signal that some results were returned and there is no need for creating an additional record. - implicitValues.replace(projectMetadataColumn, Boolean.FALSE.toString()); - } else if (Boolean.parseBoolean(implicitValues.get(projectMetadataColumn))) { + // Sets the implicit value to null to signal that some results were returned and there is no need for creating an additional record. + implicitValues.replace(projectMetadataColumn, null); + } else if (Boolean.FALSE.toString().equals(implicitValues.get(projectMetadataColumn))) { Review comment: Values here are written using either using `Boolean.FALSE.toString()` or `Boolean.TRUE.toString()` or unset. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services