-
Notifications
You must be signed in to change notification settings - Fork 223
Hold table locks when needed during analyze #5060
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: 8.0
Are you sure you want to change the base?
Conversation
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: 8/569 tests failed ⚠.
The first 10 failing tests are:
sc_truncate
phys_rep_perf
sc_timepart
sc_timepart_logicalsc_generated
analyze_partial_index_off_generated
analyze
incremental_backup_load
truncatesc_offline_generated
e08bc34
to
9bb6316
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: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 364/569 tests failed ⚠.
The first 10 failing tests are:
phys_rep_perf
sc_timepart
yast
yast_stat4scan_generated
analyze_recover_deadlock
analyze_partial_index_off_generated
analyze
autoanalyze
temp_table_truncate_leak
sc_resume2
3320810
to
1e9a0ee
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: Success ✓.
Cbuild submission: Error ⚠.
Regression testing: 4/569 tests failed ⚠.
The first 10 failing tests are:
phys_rep_perf
sc_timepart_logicalsc_generated
incremental_backup_load
truncatesc_offline_generated
5fc9635
to
45909ef
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: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 7/569 tests failed ⚠.
The first 10 failing tests are:
sc_truncate
sc_transactional_rowlocks_generated
phys_rep_perf
sc_timepart
rem
incremental_backup_load
truncatesc_offline_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: Error ⚠.
Regression testing: 4/569 tests failed ⚠.
The first 10 failing tests are:
logfill_logput_window_generated
phys_rep_perf
incremental_backup_load
truncatesc_offline_generated
3016acf
to
7106d75
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: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 6/569 tests failed ⚠.
The first 10 failing tests are:
phys_rep_perf
sc_timepart_logicalsc_generated
analyze_partial_index_off_generated
analyze
incremental_backup_load
truncatesc_offline_generated
Signed-off-by: Morgan Douglas <[email protected]> Update test Signed-off-by: mdouglas47 <[email protected]> Add to impl Signed-off-by: mdouglas47 <[email protected]> Update Signed-off-by: mdouglas47 <[email protected]> Tweak Signed-off-by: mdouglas47 <[email protected]> Tweak Signed-off-by: mdouglas47 <[email protected]> Change name of tunable Signed-off-by: mdouglas47 <[email protected]> Tweak Signed-off-by: mdouglas47 <[email protected]> Tweak Signed-off-by: mdouglas47 <[email protected]> Tweak Signed-off-by: mdouglas47 <[email protected]> Revert Signed-off-by: mdouglas47 <[email protected]> Tweak Signed-off-by: mdouglas47 <[email protected]> Tweak Signed-off-by: mdouglas47 <[email protected]> Tweak Signed-off-by: mdouglas47 <[email protected]> Add expected output for test Signed-off-by: mdouglas47 <[email protected]> Use curtran Signed-off-by: mdouglas47 <[email protected]> Only use curtran if possible Signed-off-by: mdouglas47 <[email protected]> Tweak Signed-off-by: mdouglas47 <[email protected]> Tweak Signed-off-by: mdouglas47 <[email protected]> Consistent naming Signed-off-by: mdouglas47 <[email protected]> Use existing function Signed-off-by: mdouglas47 <[email protected]> Tweak Signed-off-by: mdouglas47 <[email protected]> source file Signed-off-by: mdouglas47 <[email protected]>
7106d75
to
fde1743
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: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 6/569 tests failed ⚠.
The first 10 failing tests are:
phys_rep_perf
sc_timepart
sc_timepart_logicalsc_generated
nightmare_recovery
incremental_backup_load
truncatesc_offline_generated
The changes in this PR guard
dbtable
access with a table lock