-
Notifications
You must be signed in to change notification settings - Fork 209
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
Abort on commit if this lockid previously received deadlock #4234
base: main
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.
Lint check:
@@ db/db_tunables.h @@
+ Please update the tunable documentation (docs/pages/config/config_files.md) accordingly.
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: Success ✓.
Smoke testing: Success ✓.
Cbuild submission: Error ⚠.
Regression testing: 377/520 tests failed ⚠.
The first 10 failing tests are:
sc_truncate_lockorder_generated
systable_locking
queuedb_rollover
yast_stat4scan_generated
yast
prepare
analyze_recover_deadlock
analyze_partial_index_off_generated
autoanalyze
analyze
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.
Lint check:
@@ db/db_tunables.h @@
+ Please update the tunable documentation (docs/pages/config/config_files.md) accordingly.
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: Success ✓.
Smoke testing: Success ✓.
Cbuild submission: Error ⚠.
Regression testing: 9/520 tests failed ⚠.
The first 10 failing tests are:
delfile_block [setup failure]
sc_truncate_lockorder_generated
schemalk_extralock_generated
sc_transactional_rowlocks_generated
sc_transactional_logicalsc_generated
queuedb_rollover
prepare
lostwrite
truncatesc
3202304
to
60af6be
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.
Lint check:
@@ db/db_tunables.h @@
+ Please update the tunable documentation (docs/pages/config/config_files.md) accordingly.
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: Success ✓.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 6/520 tests failed ⚠.
The first 10 failing tests are:
sc_timepart_logicalsc_generated
sc_timepart
sc_truncate_lockorder_generated
truncatesc
schemalk_extralock_generated
sc_transactional_rowlocks_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.
Lint check:
@@ db/db_tunables.h @@
+ Please update the tunable documentation (docs/pages/config/config_files.md) accordingly.
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: 0/0 tests failed ⚠.
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.
Lint check:
@@ db/db_tunables.h @@
+ Please update the tunable documentation (docs/pages/config/config_files.md) accordingly.
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: Success ✓.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 5/520 tests failed ⚠.
The first 10 failing tests are:
sc_timepart_logicalsc_generated
sc_timepart
sc_truncate_lockorder_generated
sc_truncate
systable_locking
04f27b1
to
c1718af
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.
Lint check:
@@ db/db_tunables.h @@
+ Please update the tunable documentation (docs/pages/config/config_files.md) accordingly.
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: Success ✓.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 5/520 tests failed ⚠.
The first 10 failing tests are:
sc_timepart_logicalsc_generated
sc_timepart
sc_truncate
transchunk_manyins
sc_downgrade
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.
Lint check:
@@ db/db_tunables.h @@
+ Please update the tunable documentation (docs/pages/config/config_files.md) accordingly.
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: Success ✓.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 6/521 tests failed ⚠.
The first 10 failing tests are:
sc_timepart_logicalsc_generated
schemalk
sc_transactional
queuedb_rollover_noroll1_generated
queuedb_rollover
sc_downgrade
ead9f0f
to
42a1006
Compare
ba2fcba
to
4b23fc1
Compare
Fix various deadlock-then-commit bugs Add commit-after-deadlock test Signed-off-by: Mark Hannum <mhannum72@gmail.com>
Abort the master if it attempts to commit a transaction which has previously received a deadlock.