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

[BUGFIX] SQL dialect doesn't register for BigQuery for V2 #3324

Merged

Conversation

talagluck
Copy link
Contributor

Changes proposed in this pull request:

  • In PR [FEATURE] Enable Expectations tests for BigQuery #3219, BigQuery was added to a list of officially included and supported dialects of sqlalchemy in SqlAlchemyDataset. As per this page, BigQuery is considered an external dialect. This was leading to an issue where for BigQuery-dialected SqlAlchemyDatasets, the dialect was being set as None.

This reverts that change. I believe this resolves both #3308 and #3261 .

Definition of Done

Please delete options that are not relevant.

Thank you for submitting!

@netlify
Copy link

netlify bot commented Aug 27, 2021

✔️ Deploy Preview for niobium-lead-7998 ready!

🔨 Explore the source changes: 0bdcdfd

🔍 Inspect the deploy log: https://app.netlify.com/sites/niobium-lead-7998/deploys/61292a7d5586be000896fe1b

😎 Browse the preview: https://deploy-preview-3324--niobium-lead-7998.netlify.app

@github-actions
Copy link
Contributor

HOWDY! This is your friendly 🤖 CHANGELOG bot 🤖

Please don't forget to add a clear and succinct description of your change under the Develop header in docs_rtd/changelog.rst, if applicable. This will help us with the release process. See the Contribution checklist in the Great Expectations documentation for the type of labels to use!

Thank you!

Copy link
Contributor

@Shinnnyshinshin Shinnnyshinshin left a comment

Choose a reason for hiding this comment

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

oh my goodness. Thank you so much for catching this @talagluck. You're absolutely right 😅

@talagluck talagluck merged commit 21674f1 into develop Aug 27, 2021
@talagluck talagluck deleted the bugfix/GREAT-166/fix_bigquery_sql_dialect_issue_for_v2 branch August 27, 2021 20:52
Shinnnyshinshin pushed a commit that referenced this pull request Aug 30, 2021
…tations/great_expectations into MAINTENANCE/GDOC-219/tests_datasource_sparkdf_pandas

* 'MAINTENANCE/GDOC-219' of https://github.com/great-expectations/great_expectations:
  [MAINTENANCE] Tests for RuntimeDataConnector at DataContext-level (#3304)
  [BUGFIX] SQL dialect doesn't register for BigQuery for V2 (#3324)
  [WIP] [FEATURE] add backend args to run_diagnostics (#3257)
Shinnnyshinshin pushed a commit to jdimatteo/great_expectations that referenced this pull request Aug 31, 2021
…matteo/great_expectations into feature/add_performance_test

* 'feature/add_performance_test' of https://github.com/jdimatteo/great_expectations:
  [MAINTENANCE] Tests for RuntimeDataConnector at Datasource-level (Spark and Pandas) (great-expectations#3318)
  [MAINTENANCE] Tests for RuntimeDataConnector at DataContext-level (great-expectations#3304)
  [BUGFIX] SQL dialect doesn't register for BigQuery for V2 (great-expectations#3324)
  [WIP] [FEATURE] add backend args to run_diagnostics (great-expectations#3257)
  Release Prep release-prep-2021-08-26 (great-expectations#3320)
  Docs] hide stubbed core skills (great-expectations#3316)
  [MAINTENANCE] Write integration/E2E tests for both `GCSDataConnectors` (great-expectations#3301)
  [DOCS] Standardize capitalization of various technologies in `docs` (great-expectations#3312)
  [DOCS] Fix misc errors in "How to create renderers for Custom Expectations" (great-expectations#3315)
  docs: Remove misc TODOs to tidy up docs (great-expectations#3313)
  [DOCS] GDOC-217 remove stub links (great-expectations#3314)
  [FEATURE] Enable `GCS DataConnector` integration with `PandasExecutionEngine` (great-expectations#3264)
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.

None yet

2 participants