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

fix(pyspark): generate IF NOT EXISTS if force=True #10782

Merged
merged 2 commits into from
Feb 4, 2025

Conversation

gforsyth
Copy link
Member

@gforsyth gforsyth commented Feb 3, 2025

Description of changes

We were using the wrong keyword in sqlglot for this so it was being
swallowed silently.

Issues closed

xref #10749

@github-actions github-actions bot added tests Issues or PRs related to tests pyspark The Apache PySpark backend labels Feb 3, 2025
@cpcloud cpcloud added this to the 10.0 milestone Feb 4, 2025
@cpcloud cpcloud added bug Incorrect behavior inside of ibis ddl Issues related to creating or altering data definitions labels Feb 4, 2025
gforsyth and others added 2 commits February 4, 2025 06:34
We were using the wrong keyword in `sqlglot` for this so it was being
swallowed silently.
@cpcloud
Copy link
Member

cpcloud commented Feb 4, 2025

Looks like we're making this same mistake in MySQL as well. I'll PR that in a sec.

@cpcloud
Copy link
Member

cpcloud commented Feb 4, 2025

@gforsyth Does this fix the issue you xref'd? Or are you just noting that they are related? If it does fix it, let's add the Fixes bit so that it gets closed when this PR gets merged.

@cpcloud cpcloud merged commit c7d1d38 into ibis-project:main Feb 4, 2025
87 checks passed
@gforsyth gforsyth deleted the pyspark_exists branch February 4, 2025 12:24
@gforsyth
Copy link
Member Author

gforsyth commented Feb 4, 2025

No, there's something else going on in that issue but it's why I came across this bug

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Incorrect behavior inside of ibis ddl Issues related to creating or altering data definitions pyspark The Apache PySpark backend tests Issues or PRs related to tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants