dylanhz commented on PR #27720: URL: https://github.com/apache/flink/pull/27720#issuecomment-4073748716
> > good that we now don't have changes in `Parser.jj` however there are still changes in Calcite classes and if I see there are no corresponding changes in Calcite and even more there is no any PR to add them to Calcite which is not good in my opinion... > > Hi @snuyanzin, thanks for the review. > > I've updated the commit to avoid any modifications to the Calcite-forked classes. Could you take another look and let me know if this is what you had in mind? If not, I'd appreciate some clarification on what you'd expect. @snuyanzin PTAL when you are available, thanks! -- 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]
