Skip to content

fix: stage_sequence parameter#436

Merged
tkiehn merged 3 commits intoScalefreeCOM:mainfrom
nancypham:433-bug-invalid-identifier-edwsequence
Mar 27, 2026
Merged

fix: stage_sequence parameter#436
tkiehn merged 3 commits intoScalefreeCOM:mainfrom
nancypham:433-bug-invalid-identifier-edwsequence

Conversation

@nancypham
Copy link
Copy Markdown
Contributor

@nancypham nancypham commented Mar 12, 2026

Description

https://github.com/ScalefreeCOM/datavault4dbt/releases/tag/v1.9.12 introduced a breaking change for the sequence parameter.

Fixes #433

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.

Test Configuration:

  • datavault4dbt-Version: v1.11.0
  • dbt-Version: core v1.8.8
  • dbt-adapter-Version: Snowflake v1.8.3

Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation or included information that needs updates (e.g. in the Wiki)

@nancypham nancypham changed the title Update stage_sequence parameter fix: stage_sequence parameter Mar 13, 2026
@tkirschke tkirschke added the testing To trigger the automated test workflow as internal User. label Mar 26, 2026
@tkiehn tkiehn requested review from tkiehn March 27, 2026 13:06
@tkiehn tkiehn added testing To trigger the automated test workflow as internal User. and removed testing To trigger the automated test workflow as internal User. labels Mar 27, 2026
@remoteworkflow

This comment was marked as outdated.

@remoteworkflow remoteworkflow Bot removed the testing To trigger the automated test workflow as internal User. label Mar 27, 2026
Copy link
Copy Markdown
Collaborator

@tkiehn tkiehn left a comment

Choose a reason for hiding this comment

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

Hi @nancypham,

thanks for reporting and fixing this bug!

We will publish this with the next release.

One important thing to note if you are using the sequence-parameter: We are planning to remove this parameter with the upcoming version 2.0.0.
If you need to keep the same functionality you can instead use a derived column to rename your source-sequence-column to edwSequence.

Best Regards
Theo

@tkiehn tkiehn merged commit 5cfd8b0 into ScalefreeCOM:main Mar 27, 2026
1 check passed
@nancypham
Copy link
Copy Markdown
Contributor Author

Thanks Theo! Do you have a timeline for when next release will happen?

Noted on the sequence parameter change for v2.0.0.

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.

[BUG] "Invalid identifier edwSequence" in satellite models

4 participants