> On May 29, 2015, 2:47 p.m., Aman Sinha wrote: > > exec/java-exec/src/test/java/org/apache/drill/TestUnionAll.java, line 540 > > <https://reviews.apache.org/r/34528/diff/3/?file=973708#file973708line540> > > > > For the plan validation for filter push-down it is not sufficient to > > check for exclusions. Could you check for the filter presence on both side > > of the Union ? Otherwise we won't know if the filter somehow got dropped.
The new patch added the expected pattern for every test case - Sean Hsuan-Yi ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34528/#review85724 ----------------------------------------------------------- On May 29, 2015, 8:05 p.m., Sean Hsuan-Yi Chu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34528/ > ----------------------------------------------------------- > > (Updated May 29, 2015, 8:05 p.m.) > > > Review request for drill, Aman Sinha and Jinfeng Ni. > > > Bugs: DRILL-2746 > https://issues.apache.org/jira/browse/DRILL-2746 > > > Repository: drill-git > > > Description > ------- > > Add two DrillRules to push Project and Filter below set operators > > > Diffs > ----- > > > exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillProjectSetOpTransposeRule.java > PRE-CREATION > > exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillRuleSets.java > f7cfbf4 > exec/java-exec/src/test/java/org/apache/drill/TestUnionAll.java 5f98d90 > > exec/java-exec/src/test/resources/testframework/testUnionAllQueries/testProjectDownOverUnionAllImplicitCasting.tsv > PRE-CREATION > > exec/java-exec/src/test/resources/testframework/testUnionAllQueries/testProjectFiltertPushDownOverUnionAll.tsv > PRE-CREATION > > exec/java-exec/src/test/resources/testframework/testUnionAllQueries/testProjectPushDownOverUnionAllWithProject.tsv > PRE-CREATION > > exec/java-exec/src/test/resources/testframework/testUnionAllQueries/testProjectPushDownOverUnionAllWithoutProject.tsv > PRE-CREATION > > exec/java-exec/src/test/resources/testframework/testUnionAllQueries/testProjectPushDownProjectColumnReorderingAndAlias.tsv > PRE-CREATION > > exec/java-exec/src/test/resources/testframework/testUnionAllQueries/testProjectWithExpressionPushDownOverUnionAll.tsv > PRE-CREATION > > Diff: https://reviews.apache.org/r/34528/diff/ > > > Testing > ------- > > Unit test, tpch, fucntional > > > Thanks, > > Sean Hsuan-Yi Chu > >
