Github user vdiravka commented on the issue:
https://github.com/apache/drill/pull/1152
+1
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1152
Thanks, @chunhui-shi - marked it as ready-to-commit since the original
feature was already merged to 1.13. The batch committer this week can take
another look as well.
---
Github user chunhui-shi commented on the issue:
https://github.com/apache/drill/pull/1152
+1, good to me.
---
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1152
@HanumathRao, @chunhui-shi any more comments from you?
---
Github user arina-ielchiieva commented on the issue:
https://github.com/apache/drill/pull/1152
@HanumathRao thanks for the review. Applied code review comment.
---
Github user arina-ielchiieva commented on the issue:
https://github.com/apache/drill/pull/1152
@chunhui-shi yes, you are correct, we are trying to find item star columns
and push them into scan
This case if F is optional then we don't have filter and there will no
filter push
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/1152
@chunhui-shi or @HanumathRao can you please review this?
---
Github user arina-ielchiieva commented on the issue:
https://github.com/apache/drill/pull/1152
@chunhui-shi please review.
---