Skip to content

sql_heuristics_aliases #1225

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

Merged
merged 20 commits into from
May 6, 2025
Merged

sql_heuristics_aliases #1225

merged 20 commits into from
May 6, 2025

Conversation

jgaleotti
Copy link
Collaborator

first version of SQL aliases for advanced SQL heuristics

@jgaleotti jgaleotti requested a review from arcuri82 April 16, 2025 17:52
@arcuri82
Copy link
Collaborator

@jgaleotti hi. you made some changes after you requested my review, and now the build fails. please fix, and re-ask my review once done ;)

@jgaleotti
Copy link
Collaborator Author

@jgaleotti hi. you made some changes after you requested my review, and now the build fails. please fix, and re-ask my review once done ;)

Thanks, I'm into this.

@jgaleotti jgaleotti closed this Apr 21, 2025
@jgaleotti jgaleotti reopened this Apr 21, 2025
jgaleotti added 6 commits May 5, 2025 10:05
…re used instead of String values for table and column identifiers
…o give backward compatibility with HeuristicsCalculator)
…euristics, while SqlHeuristicsCalculator is targeted for the new/complete Sql heuristics.
@jgaleotti jgaleotti requested a review from arcuri82 May 6, 2025 11:18
@arcuri82 arcuri82 merged commit 0613970 into master May 6, 2025
14 checks passed
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.

2 participants