Github user amansinha100 commented on the issue:
https://github.com/apache/drill/pull/910
+1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feat
Github user sohami commented on the issue:
https://github.com/apache/drill/pull/910
+1 LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the featu
Github user arina-ielchiieva commented on the issue:
https://github.com/apache/drill/pull/910
@sohami addressed code review comments. Please review when possible.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your pr
Github user arina-ielchiieva commented on the issue:
https://github.com/apache/drill/pull/910
@sohami PR is updated, information about new approach is added in Jira.
Please review when possible.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user sohami commented on the issue:
https://github.com/apache/drill/pull/910
Had a chat with Arina on this and it is not the final PR, she will be
updating the PR with new set of changes. @arina-ielchiieva - It would be great
if you can put some context related to new set of ch
Github user paul-rogers commented on the issue:
https://github.com/apache/drill/pull/910
@sohami, @sindhurirayavaram can you review this as it is close to the areas
you've been working on.
---
If your project is set up for it, you can reply to this email and have your
reply appear on