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(snowflake): use as_posix instead of as_uri to avoid escaping special characters #10792

Merged
merged 2 commits into from
Feb 5, 2025

Conversation

cpcloud
Copy link
Member

@cpcloud cpcloud commented Feb 5, 2025

Fixes failing Snowflake CI

@cpcloud cpcloud added this to the 10.0 milestone Feb 5, 2025
@cpcloud cpcloud added the ci-run-cloud Run BigQuery, Snowflake, Databricks, and Athena backend tests label Feb 5, 2025
@ibis-docs-bot ibis-docs-bot bot removed the ci-run-cloud Run BigQuery, Snowflake, Databricks, and Athena backend tests label Feb 5, 2025
@github-actions github-actions bot added tests Issues or PRs related to tests snowflake The Snowflake backend labels Feb 5, 2025
@cpcloud cpcloud force-pushed the fix-snowflake-put branch 2 times, most recently from d085511 to b4e64b9 Compare February 5, 2025 13:14
@github-actions github-actions bot added the ci Continuous Integration issues or PRs label Feb 5, 2025
@cpcloud cpcloud added the ci-run-cloud Run BigQuery, Snowflake, Databricks, and Athena backend tests label Feb 5, 2025
@ibis-docs-bot ibis-docs-bot bot removed the ci-run-cloud Run BigQuery, Snowflake, Databricks, and Athena backend tests label Feb 5, 2025
@cpcloud cpcloud changed the title test(snowflake): quote all PUT local path inputs fix(snowflake): use as_posix instead of as_uri to avoid escaping special characters Feb 5, 2025
@github-actions github-actions bot added the clickhouse The ClickHouse backend label Feb 5, 2025
@cpcloud cpcloud merged commit ec41d4c into ibis-project:main Feb 5, 2025
87 checks passed
@cpcloud cpcloud deleted the fix-snowflake-put branch February 5, 2025 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous Integration issues or PRs clickhouse The ClickHouse backend snowflake The Snowflake backend tests Issues or PRs related to tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant