Hi @gaodayue, in the meantime could you please check the tests and inspections? 
In the tests, it looks like lots of CalciteQueryTests are failing due to the 
new `sqlQueryId`. Maybe the verifier that compares the queries for correctness 
should strip `sqlQueryId` first. In the inspections, it is reporting that the 
Query.getSqlQueryId() method is never used. If it's not needed, you can remove 
it.

[ Full content available at: 
https://github.com/apache/incubator-druid/pull/6302 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to