adarshsanjeev commented on PR #13844: URL: https://github.com/apache/druid/pull/13844#issuecomment-1475906681
Thanks, @paul-rogers! I have resolved the failures and merge conflicts, the PR should be good to go. Regarding the test failures, the cause does not seem to have been introduced by a specific PR (at least not that I can tell), but some discrepancies that have always existed due to how MSQ handles it. I think there are more than these as all tests have not been ported to `CalciteSelectQueryMSQTest` yet. I have raised two issues to cover these. https://github.com/apache/druid/issues/13950 https://github.com/apache/druid/issues/13951 I can raise these in the Apache community slack to discuss how to handle these. -- 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]
