Skip to content

Conversation

@LantaoJin
Copy link
Member

@LantaoJin LantaoJin commented Mar 4, 2025

Description

Fix the join condition resolving bug introduced by IN subquery implementation
CalcitePPLJoinTest.testJoinConditionWithoutTableName() failed after merged #3371

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • New functionality has javadoc added.
  • New functionality has a user manual doc added.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff.
  • Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

…ry impl

Signed-off-by: Lantao Jin <ltjin@amazon.com>
@LantaoJin LantaoJin marked this pull request as ready for review March 4, 2025 11:51
@LantaoJin LantaoJin added bug Something isn't working calcite calcite migration releated labels Mar 4, 2025
@LantaoJin
Copy link
Member Author

cc @qianheng-aws @penghuo

@LantaoJin LantaoJin merged commit 2591b15 into opensearch-project:feature/calcite-engine Mar 5, 2025
7 of 15 checks passed
penghuo pushed a commit to penghuo/os-sql that referenced this pull request Mar 12, 2025
penghuo pushed a commit that referenced this pull request Jun 16, 2025
…ry impl (#3377)

Signed-off-by: xinyual <xinyual@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working calcite calcite migration releated

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants