Skip to content

Remove ireq from open_temp_db_resume#6017

Open
dorinhogea wants to merge 1 commit into
bloomberg:mainfrom
dorinhogea:opentmpdb-noiq
Open

Remove ireq from open_temp_db_resume#6017
dorinhogea wants to merge 1 commit into
bloomberg:mainfrom
dorinhogea:opentmpdb-noiq

Conversation

@dorinhogea

Copy link
Copy Markdown
Contributor

Schema change phase 1 (the asynchronous phase) does not need an iq objects; this is first PR to achieve that goal; removes iq from lower level function that creates a new btree-s for the table being altered.

Signed-off-by: Dorin Hogea <dhogea@bloomberg.net>
@dorinhogea

Copy link
Copy Markdown
Contributor Author

@mponomar thank you!

@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:
sc_vutf8_max_update [failed with core dumped]
sc_tableversion_logicalsc_generated [failed with core dumped]
sc_tableversion [failed with core dumped]
sc_add_vutf8_off_osql_odh_blob_generated [failed with core dumped]
sc_add_vutf8_logicalsc_generated [failed with core dumped]
sc_add_vutf8 [failed with core dumped]
sc_timepart_multiddl_generated
reco-ddlk-sql **quarantined**
consumer_non_atomic_default_consumer_generated **quarantined**
sc_transactional

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