Skip to content

Preserve table_version entry on rename to avoid reuse - #6202

Merged
dorinhogea merged 1 commit into
bloomberg:mainfrom
dorinhogea:rename-tblver-fix
Sep 18, 2026
Merged

dorinhogea merged 1 commit into
bloomberg:mainfrom
dorinhogea:rename-tblver-fix

Conversation

@dorinhogea

Copy link
Copy Markdown
Contributor

^title

@roborivers roborivers left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cbuild submission: Error ⚠.
Regression testing: Success ✓.

The first 10 failing tests are:
sc_resume_logicalsc_generated **quarantined**
tsa
consumer_non_atomic_default_consumer_generated **quarantined**
sc_downgrade [timeout] **quarantined**

Comment thread tests/rename_populate.test/runit
Comment thread tests/rename_populate.test/runit
@dorinhogea
dorinhogea force-pushed the rename-tblver-fix branch 2 times, most recently from e970cb9 to 50b494b Compare September 17, 2026 20:36

@roborivers roborivers left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cbuild submission: Success ✓.
Regression testing: Success ✓.

The first 10 failing tests are:
reco-ddlk-sql **quarantined**
consumer_non_atomic_default_consumer_generated **quarantined**
sc_downgrade [timeout] **quarantined**

Comment thread schemachange/sc_rename_table.c Outdated
Comment thread tests/rename_populate.test/runit
Signed-off-by: Dorin Hogea <dhogea@bloomberg.net>
@dorinhogea

Copy link
Copy Markdown
Contributor Author

@emelialei88 thank you

@dorinhogea
dorinhogea merged commit 26aede9 into bloomberg:main Sep 18, 2026
5 checks passed
@dorinhogea
dorinhogea deleted the rename-tblver-fix branch September 18, 2026 18:37

@roborivers roborivers left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cbuild submission: Success ✓.
Regression testing: Success ✓.

The first 10 failing tests are:
scindex_logicalsc_generated [failed with core dumped]
analyze_partial_index_off_generated [failed with core dumped] **quarantined**
analyze [failed with core dumped] **quarantined**
consumer_non_atomic_default_consumer_generated **quarantined**
datetimeusec
datetime
cdb2api
sc_downgrade [timeout] **quarantined**
reco-ddlk-sql [timeout] **quarantined**
skipscan [timeout] **quarantined**

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.

3 participants