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

fixes issue 475 MSSQL Unqualified Schema Table Parsing #530

Merged

Conversation

T-Santos
Copy link
Contributor

@T-Santos T-Santos commented Oct 24, 2024

Resolves #475

@macbre macbre self-assigned this Oct 24, 2024
@macbre macbre added the bug label Oct 24, 2024
Comment on lines +31 to +32
def test_simple_queries_tables(query, expected):
assert Parser(query).tables == expected
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding the test coverage!

@macbre
Copy link
Owner

macbre commented Oct 24, 2024

@T-Santos - thanks for the fix, merging.

@macbre macbre merged commit da144a4 into macbre:master Oct 24, 2024
11 checks passed
@T-Santos T-Santos deleted the feature/support-default-schema-notation branch October 24, 2024 15:27
@T-Santos
Copy link
Contributor Author

@macbre if you have a moment, would you be able to put out a python release so I'm able to pull the package (or inform me how I'm able to do so. I poked around a bit and didn't see any options for me to trigger a release but I certainly could have missed it). Thanks!

@macbre
Copy link
Owner

macbre commented Oct 24, 2024

Done in https://github.com/macbre/sql-metadata/releases/tag/v2.14.0

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

Successfully merging this pull request may close these issues.

Parser().tables fix for missing schema in ms_sql and sybase
2 participants