Skip to content

[FIX] Empty lines or tabs inside SQL query of view definition will trigger redeployment of databricks_sql_table resource #3865 #4003

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[FIX] Empty lines or tabs inside SQL query of view definition will trigger redeployment of databricks_sql_table resource #3865 #4003

wants to merge 1 commit into from
+29 −0

Conversation

p-venclovas
Copy link

Changes

Added formatting for new lines and tabs. This formatting is already applied on databricks side and it shows up when terraform read the state of view definition.

Tests

  • make test run locally
  • relevant acceptance tests are passing
  • using Go SDK

@p-venclovas p-venclovas requested review from a team as code owners September 6, 2024 12:59
@p-venclovas p-venclovas requested review from mgyucht and removed request for a team September 6, 2024 12:59
@p-venclovas p-venclovas mentioned this pull request Sep 6, 2024
Open
@github-actions GitHub Actions
Copy link

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/terraform

Inputs:

  • PR number: 4003
  • Commit SHA: 0957f4445d2916dc2dab7d12d491f47697eb33f1

Checks will be approved automatically on success.

@github-actions GitHub Actions
Copy link

Please ensure that the NEXT_CHANGELOG.md file is updated with any relevant changes.
If this is not necessary for your PR, please include the following in your PR description:
NO_CHANGELOG=true
and rerun the job.

@nkvuong nkvuong deployed to test-trigger-is June 26, 2025 08:46 — with GitHub Actions Active
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mgyucht mgyucht Awaiting requested review from mgyucht mgyucht is a code owner automatically assigned from databricks/eng-plat-auto-exp-reviewers

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@p-venclovas @nkvuong