reproduce and fix race condition push_clnt_cols, and l_send_back_row - #6087
reproduce and fix race condition push_clnt_cols, and l_send_back_row#6087rishitha957 wants to merge 1 commit into
Conversation
| return luaL_error(L, "attempt to change number of columns"); | ||
| } | ||
| if (num_cols < parent->ntypes) { | ||
| for (int i = num_cols - 1; i < parent->ntypes; ++i) { |
There was a problem hiding this comment.
this is an existing bug (i think this is when we shrink columns to a lower count, we should free only from num_cols .. )
roborivers
left a comment
roborivers
left a comment
There was a problem hiding this comment.
Cbuild submission: Success ✓.
Regression testing: Success ✓.
The first 10 failing tests are:
sc_truncate_multiddl_generated [db unavailable at finish] **quarantined**
comdb2sys_pagesize_generated [db unavailable at finish]
sc_resume_logicalsc_generated **quarantined**
sc_resume
consumer_non_atomic_default_consumer_generated **quarantined**
tunables
sc_downgrade [timeout] **quarantined**
sp_twofiles_generated [timeout]
sp_snapshot_generated [timeout]
sp_queueodh_generated [timeout]
Signed-off-by: Rishitha Kalicheti <rkalicheti1@bloomberg.net> fix failing tests, add tunable to the tunables.test Signed-off-by: Rishitha Kalicheti <rkalicheti1@bloomberg.net>
roborivers
left a comment
roborivers
left a comment
There was a problem hiding this comment.
Cbuild submission: Success ✓.
Regression testing: Success ✓.
The first 10 failing tests are:
consumer_non_atomic_default_consumer_generated **quarantined**
sc_downgrade [timeout] **quarantined**
2 participants
No description provided.