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_r392708180
########## File path: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/FilePushDownFilter.java ########## @@ -244,4 +247,15 @@ protected void doOnMatch(RelOptRuleCall call, FilterPrel filter, ProjectPrel pro final RelNode newFilter = filter.copy(filter.getTraitSet(), Collections.singletonList(newNode)); call.transformTo(newFilter); } + + /** + * Checks whether specified group scan supports filter push down. + * + * @param groupScan group scan to check + * @return {@code true} if specified group scan supports filter push down, {@code false} otherwise + */ + private static boolean supportsFilterPushDown(GroupScan groupScan) { + return groupScan instanceof AbstractGroupScanWithMetadata Review comment: Good idea, thanks! Added such method into `GroupScan`. ---------------------------------------------------------------- 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