Github user arina-ielchiieva commented on the issue:

    https://github.com/apache/drill/pull/996
  
    @paul-rogers thanks for approving pull request but actually before merging 
these changes in Drill (since I am not still convinced we need them), I was 
expecting @HanumathRao to do investigation if newer Calcite versions handle 
this case, if yes, maybe postpone this changes till the Calcite upgrade or 
cherry-pick them in Drill Calcite (if urgent). If not, create Jira in Calcite 
project and provide fix, and maybe cherry-pick this very fix once done (again 
if urgent). Since if all these checks can be done in Calcite we don't need to 
customize Drill.


---

Reply via email to