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

Reply via email to