[GitHub] [drill] vvysotskyi commented on pull request #2733: DRILL-8380: Remove customised SqlValidatorImpl.deriveAlias

2023-01-16 Thread GitBox 


vvysotskyi commented on PR #2733:
URL: https://github.com/apache/drill/pull/2733#issuecomment-1384890894

   Yes, it can be merged before Calcite is released.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@drill.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [drill] vvysotskyi commented on pull request #2733: DRILL-8380: Remove customised SqlValidatorImpl.deriveAlias

2023-01-16 Thread GitBox 


vvysotskyi commented on PR #2733:
URL: https://github.com/apache/drill/pull/2733#issuecomment-1384305483

   I think Splunk tests have somewhere a condition to fail if I'm the author of 
the commit 
   Here is CI run for another my commit in the master branch that has the same 
error: https://github.com/apache/drill/actions/runs/3874925191/jobs/6620797212


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@drill.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org