luocooong commented on a change in pull request #2436:
URL: https://github.com/apache/drill/pull/2436#discussion_r793295277
##########
File path:
contrib/storage-mongo/src/main/java/org/apache/drill/exec/store/mongo/plan/MongoPluginImplementor.java
##########
@@ -292,7 +297,11 @@ public GroupScan getPhysicalOperator() {
newSpec, columns, runAggregate);
}
- public static int direction(RelFieldCollation fieldCollation) {
+ private boolean hasMongoGroupScan(RelNode node) {
Review comment:
@vvysotskyi I have an idea, could we transform that function as a base
or interface function (such the `hasXXXGroupscan()`)? Ensure the new format
based on this framework is needed to implement this filter function.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
