Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Planner rules were not configured correctly in CalciteSelectJoinQueryMSQTest.SortMerge #16857

Open
kgyrtkirk opened this issue Aug 7, 2024 · 0 comments

Comments

@kgyrtkirk
Copy link
Member

A small refactor of the CalciteSelectJoinQueryMSQTest class (in #16798) have uncovered that the way joinAlgorithm was set earlier - did not have given any chance to the CalcitePlanner to enable rules according to that setting.

The refactor changed the approach to use the queryContext to set joinAlgorithm - which have caused two SortMerge testcases fail:

   public void testFilterAndGroupByLookupUsingJoinOperatorWithNotFilter(Map<String, Object> queryContext)
   public void testRegressionFilteredAggregatorsSubqueryJoins(Map<String, Object> queryContext)

These will be disabled with an assume by the above mentioned PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant