fix partition rollout and remove scdone outside finalizing txn #4367
Conversation
There was a problem hiding this comment.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 374/528 tests failed ⚠.
The first 10 failing tests are:
simple_timepart_reptimeout_generated
timepart_changeretention
timepart_retention1
simple_timepart
sc_timepart_logicalsc_generated
sc_timepart
timepart_auth
renametable
renametable_lightweight_generated
timepart_readonly
dorinhogea
force-pushed
the
fix_lock_inversion_partitionsm
branch
from
373084f
to
0536b84
Compare
There was a problem hiding this comment.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 447/529 tests failed ⚠.
The first 10 failing tests are:
systable_locking
timepart_retention1
timepart_changeretention
simple_timepart_reptimeout_generated
simple_timepart
sc_timepart_logicalsc_generated
sc_resume_logicalsc_generated
sc_resume
sc_timepart
timepart_auth
There was a problem hiding this comment.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Error ⚠.
Regression testing: 436/529 tests failed ⚠.
The first 10 failing tests are:
systable_locking
timepart_retention1
simple_timepart_reptimeout_generated
timepart_changeretention
simple_timepart
sc_timepart
sc_resume_logicalsc_generated
sc_timepart_logicalsc_generated
sc_resume
queuedb_rollover
dorinhogea
force-pushed
the
fix_lock_inversion_partitionsm
branch
from
0536b84
to
8238931
Compare
There was a problem hiding this comment.
Coding style check: Error. ⚠.
Smoke testing: Error ⚠.
Cbuild submission: Error ⚠.
Regression testing: 0/0 tests failed ⚠.
There was a problem hiding this comment.
Coding style check: Error. ⚠.
Smoke testing: Error ⚠.
Cbuild submission: Error ⚠.
Regression testing: 396/532 tests failed ⚠.
The first 10 failing tests are:
truncatesc
sc_resume_logicalsc_generated
sc_resume
queuedb_rollover_noroll1_generated
timepart_auth
timepart_changeretention
renametable_lightweight_generated
renametable
timepart_readonly
timepart_constraints
dorinhogea
force-pushed
the
fix_lock_inversion_partitionsm
branch
from
8238931
to
9674505
Compare
dorinhogea
force-pushed
the
fix_lock_inversion_partitionsm
branch
from
9674505
to
99cda01
Compare
There was a problem hiding this comment.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 5/532 tests failed ⚠.
The first 10 failing tests are:
schemalk_logicalsc_generated
sc_transactional_rowlocks_generated
timepart_trunc
comdb2sys_queueodh_generated
truncatesc_offline_generated
dorinhogea
force-pushed
the
fix_lock_inversion_partitionsm
branch
2 times, most recently
from
d8e9572
to
2e81e41
Compare
There was a problem hiding this comment.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 396/532 tests failed ⚠.
The first 10 failing tests are:
replay_history
truncatesc
sc_resume_logicalsc_generated
sc_resume
timepart_auth
timepart_changeretention
renametable_lightweight_generated
renametable
timepart_readonly
timepart_constraints
There was a problem hiding this comment.
Coding style check: Error. ⚠.
Smoke testing: Error ⚠.
Cbuild submission: Success ✓.
Regression testing: 0/0 tests failed ⚠.
dorinhogea
force-pushed
the
fix_lock_inversion_partitionsm
branch
from
2e81e41
to
99cda01
Compare
There was a problem hiding this comment.
Coding style check: Error. ⚠.
Smoke testing: Error ⚠.
Cbuild submission: Success ✓.
Regression testing: 0/0 tests failed ⚠.
dorinhogea
force-pushed
the
fix_lock_inversion_partitionsm
branch
from
99cda01
to
fc95886
Compare
There was a problem hiding this comment.
Coding style check: Error. ⚠.
Smoke testing: Error ⚠.
Cbuild submission: Error ⚠.
Regression testing: 0/0 tests failed ⚠.
There was a problem hiding this comment.
Coding style check: Error. ⚠.
Smoke testing: Error ⚠.
Cbuild submission: Error ⚠.
Regression testing: 0/0 tests failed ⚠.
avoid reading llmeta during views update (lock inversion prevention) remove code and tunable sc_done_same_tran (only on going forward) fix scdone for comdb2sc and partition rollouts, where it is sent outside finalizing txn Signed-off-by: Dorin Hogea <[email protected]>
dorinhogea
force-pushed
the
fix_lock_inversion_partitionsm
branch
from
fc95886
to
f3fc827
Compare
There was a problem hiding this comment.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 9/534 tests failed ⚠.
The first 10 failing tests are:
sc_resume
schemalk_extralock_generated
schemalk
phys_rep_tiered_nosource_generated
phys_rep_tiered
transchunk_manyins
comdb2sys
sc_downgrade
truncatesc_offline_generated
|
resubmitted with shorted name because it breaks robom (#4457) |
3 things: