TuroczyX commented on PR #4058: URL: https://github.com/apache/hive/pull/4058#issuecomment-1495702194
Sorry for dumb question but where do we use antlr? It is in the query analyzer or in the parser? Also what is the purpose of this library here? (I'm not familiar with that and I'm asking because I'm curious :) ) Does this change introduce any breaking change? As I see from the antlr page it is 20% faster the 4 then the previous which is nice. Also in the hive 4 (4.1?) we are able to introduce if it won't introduce breaking change. But It would be great to have an understating of the risks and motivation. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
