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_r395145166
########## File path: exec/java-exec/src/main/java/org/apache/drill/exec/metastore/store/SimpleFileTableMetadataProvider.java ########## @@ -97,47 +100,38 @@ public FileMetadata getFileMetadata(Path location) { @Override public NonInterestingColumnsMetadata getNonInterestingColumnsMetadata() { - return null; + return new NonInterestingColumnsMetadata(Collections.emptyMap()); Review comment: Yes, that's correct that we don't track stats for such columns. This naming came from parquet metadata cache improvements and left to be consistent with that code. ---------------------------------------------------------------- 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