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

Update Databricks to require default catalog. #1566

Merged
merged 3 commits into from Aug 14, 2023

Conversation

mknowlton89
Copy link
Collaborator

@mknowlton89 mknowlton89 commented Aug 11, 2023

Features and Changes

An organization using a Databricks data source is unable to use GrowthBook's Schema Browser as the query to generate the schema browser requires the catalog to be prefixed to the beginning of the FROM statement.

Testing

  • Attempted to connect to our sample Databricks database without the changes in this PR and ensure you get an error when trying to view the Schema Browser that states there was an error.
  • Then, with the changes in this PR, using the same credentials, update the Connection Information and include the catalog, and then confirm we're able to generate an information_schema successfully.

@github-actions
Copy link

Your preview environment pr-1566-bttf has been deployed.

Preview environment endpoints are available at:

@mknowlton89 mknowlton89 marked this pull request as ready for review August 14, 2023 10:37
@mknowlton89 mknowlton89 merged commit 4613f30 into main Aug 14, 2023
3 checks passed
@mknowlton89 mknowlton89 deleted the mk/databricks-info-schema branch August 14, 2023 12:34
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