Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/22642 )
Change subject: [util] correct invariant in ThreadPoolToken::Transition() ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/22642/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/22642/1//COMMIT_MSG@10 PS1, Line 10: State::IDLE > Is this IDLE state or RUNNING state? Ah, indeed -- the issue was about transitioning from RUNNING state, and PS1 of this patch doesn't fix the issue, rather than removing the check on the invariant. I'm planning to post a fix soon. Thank you! -- To view, visit http://gerrit.cloudera.org:8080/22642 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I6cfcdff743ad0074446ab9664dda43e4a9cb23ea Gerrit-Change-Number: 22642 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Comment-Date: Thu, 20 Mar 2025 01:41:27 +0000 Gerrit-HasComments: Yes
