Alexey Serbin has uploaded a new patch set (#3) to the change originally created by Todd Lipcon. ( http://gerrit.cloudera.org:8080/15791 )
Change subject: transaction: avoid locking for TransactionState::timestamp_ ...................................................................... transaction: avoid locking for TransactionState::timestamp_ This field is already synchronized externally for mutations by the transaction lifecycle (we never have a case where multiple threads are trying to concurrently assign a timestamp to a transaction). The timestamp might be concurrently read (eg by ToString() for the /transactions web page) but an atomic variable is sufficient for this use case. I saw a couple percent of the Apply thread's CPU going to managing this locking, which should go away with this change. Change-Id: I2c6bc8b85a066426bcf4bc2d1c058bc75c1c38ec --- M src/kudu/tablet/ops/op.cc M src/kudu/tablet/ops/op.h 2 files changed, 12 insertions(+), 11 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/91/15791/3 -- To view, visit http://gerrit.cloudera.org:8080/15791 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I2c6bc8b85a066426bcf4bc2d1c058bc75c1c38ec Gerrit-Change-Number: 15791 Gerrit-PatchSet: 3 Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Andrew Wong <andrew.w...@cloudera.com> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Volodymyr Verovkin <verjov...@cloudera.com>