{182781126} fix api_history_hung#5777
Conversation
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:
cldeadlock
consumer_non_atomic_default_consumer_generated
remsql_locks_rte_connect_generated
remsql_locks
load_cache_autocache_generated
rishitha957
force-pushed
the
api_history_hung
branch
from
0cb2396 to
9146c65
Compare
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:
scindex
disttxn
sc_resume_logicalsc_generated
consumer_non_atomic_default_consumer_generated
remsql_locks_rte_connect_generated
remsql_locks
reco-ddlk-sql
rishitha957
force-pushed
the
api_history_hung
branch
2 times, most recently
from
07a6865 to
963ccb5
Compare
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:
truncatesc_offline_generated [core dumped]
scindex_logicalsc_generated
sc_swapfields
consumer_non_atomic_default_consumer_generated
sc_transactional_rowlocks_generated
remsql_locks_rte_connect_generated
remsql_locks
reco-ddlk-sql
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
remsql_locks_rte_connect_generated
remsql_locks
sc_downgrade
truncatesc_offline_generated
reco-ddlk-sql
Signed-off-by: Rishitha Kalicheti <rkalicheti1@bloomberg.com>
rishitha957
force-pushed
the
api_history_hung
branch
from
963ccb5 to
6adeff5
Compare
riverszhang89
left a comment
riverszhang89
left a comment
There was a problem hiding this comment.
The over-design around rwlock is also removed in this patch. This is very simple and clean now. LGTM.
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:
nogensc [setup failure]
scindex
scindex_logicalsc_generated
consumer_non_atomic_default_consumer_generated
unifiedcancel
reco-ddlk-sql
4 participants
fixes dangling state of curr/iter when get_next_entry is called on a hash, uses hash_for instead.
uses ctx for collecting api_history entries of node/task of clientstats.