Github user arina-ielchiieva commented on a diff in the pull request:
https://github.com/apache/drill/pull/1078#discussion_r160665449
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/partition/FindPartitionConditions.java
---
@@ -195,8 +195,16 @@ private void popOpStackAndBuildFilter() {
* For all other operators we clear the children if one of the
* children is a no push.
*/
- assert currentOp.getOp().getKind() == SqlKind.AND;
- newFilter = currentOp.getChildren().get(0);
+ if (currentOp.getOp().getKind() == SqlKind.AND) {
+ newFilter = currentOp.getChildren().get(0);
+ for(OpState opState : opStack) {
--- End diff --
Please add space: `for (`
---
