yashmayya commented on PR #16570: URL: https://github.com/apache/pinot/pull/16570#issuecomment-3173808199
@ankitsultana I had to add skip rules for the new physical query optimizer for some of the new set operation test queries added here because of duplicates in the `UNION` results. Disabling the `UnionToDistinct` Calcite rule by default fixes the issue with the new optimizer but we can't do this until the next release due to the backward incompatibility issue I've outlined in the PR description. These tests are most likely simply uncovering existing bugs in the physical optimizer because they also fail if I add them on `master` without changes from this PR. Could you please take a look whenever possible? -- 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]
