vvysotskyi commented on pull request #2275: URL: https://github.com/apache/drill/pull/2275#issuecomment-881235096
@paul-rogers, thanks for the review! These plugins should be of different types to reproduce the issue, for example, JDBC and Elasticsearch, or JDBC and Cassandra, so it is a problem for adding end-to-end test since we don't have a module with several plugin dependencies. Regarding updated rules, the main change is in its constructor, now it has the `RelOptRuleOperand` that controls conventions of the rel nodes, but it is slightly problematic to create all required instances for checking whether the rule would or wouldn't match the specific rel node. -- 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]
