Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17059
Change subject: KUDU-2612: non-zero queue size for txn-commit pool ...................................................................... KUDU-2612: non-zero queue size for txn-commit pool I found my new tests for the automatic txn participant registration often hit the following CHECK(): F0212 06:49:33.448449 611 txn_status_manager.cc:371] Check failed: _s.ok() Bad status: Service unavailable: Thread pool is at capacity (10/10 tasks running, 2/0 tasks queued) As I can see, the code in CommitTasks::ScheduleAbortTxnWrite() doesn't handle the condition of running out of the "txn-commit" pool's queue. That means the pool is supposed to have an unlimited queue size, like other pools which exhibit similar behaviour: e.g., the pool for removing tablets, the pool for opening tablets, etc. Indeed, since the number of concurrently opened multi-row transactions isn't limited, TxnStatusManager should be able handle the case when many transactions are being finalized simultaneously. This patch removes the queue size limit for the "txn-commit" pool (i.e. effectively setting it to INT_MAX), mirroring the behavior of "tablet-open" and "tablet-delete" pools. Change-Id: Idb3de2fd41936862eec8f2616096db16ff86c070 --- M src/kudu/tserver/ts_tablet_manager.cc 1 file changed, 0 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/59/17059/1 -- To view, visit http://gerrit.cloudera.org:8080/17059 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Idb3de2fd41936862eec8f2616096db16ff86c070 Gerrit-Change-Number: 17059 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
