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

Alias mismatch bugfix #80

Open
wants to merge 1 commit into
base: li-1.21.0
Choose a base branch
from
Open

Conversation

aastha25
Copy link
Contributor

@aastha25 aastha25 commented Feb 27, 2023

Applied various commits from upstream to backport a fix which appends the correct alias in projected fields.

Commits for reference:

[CALCITE-3593] JDBC adapter generates incorrect HAVING clause for BigQuery (Jin Xing)
apache/calcite@3621b61

https://issues.apache.org/jira/browse/CALCITE-3751
apache/calcite@3c9e156

apache/calcite@76ec797
Add ignoreClauses flag in builder and update visit(filter e) to use that in RelToSqlC & PigRelToSqlC

[CALCITE-3936] apache/calcite#2047
apache/calcite@8bfc380

Testing:
sh-3.2$ ./mvnw clean install -Dgpg.skip -Dcheckstyle.skip -f core/pom.xml
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 05:45 min
[INFO] Finished at: 2023-02-28T05:12:23-08:00
[INFO] ------------------------------------------------------------------------

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

Successfully merging this pull request may close these issues.

1 participant