-
Notifications
You must be signed in to change notification settings - Fork 229
Fix bugs in async partitioned table merge resume #5240
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
base: main
Are you sure you want to change the base?
Conversation
c0f0d15 to
f4ee997
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Error. ⚠.
Smoke testing: Error ⚠.
Cbuild submission: Error ⚠.
Regression testing: 0/0 tests failed ⚠.
fc25ea4 to
aa09314
Compare
Signed-off-by: Morgan Douglas <[email protected]>
Signed-off-by: mdouglas47 <[email protected]>
Signed-off-by: mdouglas47 <[email protected]>
- Improve readability - Test that we don't lose writes that happen during the resume Signed-off-by: mdouglas47 <[email protected]>
Signed-off-by: mdouglas47 <[email protected]>
Signed-off-by: mdouglas47 <[email protected]>
30d90cc to
182da5e
Compare
Signed-off-by: mdouglas47 <[email protected]>
Signed-off-by: mdouglas47 <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Error. ⚠.
Smoke testing: Error ⚠.
Cbuild submission: Error ⚠.
Regression testing: 597/616 tests failed ⚠.
The first 10 failing tests are:
scindex
rebuild_table_options
sc_constraints_logicalsc_generated
sc_constraints
incremental_backup
truncatesc
truncatesc_offline_generated
maxtable
odh_blobs
scindex_logicalsc_generated
Signed-off-by: mdouglas47 <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Error. ⚠.
Smoke testing: Error ⚠.
Cbuild submission: Success ✓.
Regression testing: 599/616 tests failed ⚠.
The first 10 failing tests are:
scindex
cldeadlock
sc_constraints_logicalsc_generated
incremental_backup_usenames
incremental_backup
truncatesc
truncatesc_offline_generated
maxtable
odh_blobs
scindex_logicalsc_generated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 494/616 tests failed ⚠.
The first 10 failing tests are:
sc_force
scindex
sc_versmismatch_logicalsc_generated
sc_constraints
sc_resume_logicalsc_generated
sc_inserts
sc_redo
catchup
odh_blobs
sc_versmismatch
87572a9 to
86b914f
Compare
Signed-off-by: mdouglas47 <[email protected]>
Signed-off-by: mdouglas47 <[email protected]>
Signed-off-by: mdouglas47 <[email protected]>
Signed-off-by: mdouglas47 <[email protected]>
Signed-off-by: mdouglas47 <[email protected]>
Signed-off-by: mdouglas47 <[email protected]>
No description provided.