accumulo icon indicating copy to clipboard operation
accumulo copied to clipboard

Look into using a Striped lock for bulk load zookeeper checks.

Open keith-turner opened this issue 7 years ago • 4 comments

Bulk load RPCs check to see if the transaction is still alive in ZooKeeper before executing. While doing this check a tablet server wide lock is held. Therefore concurrent bulk loads executing different transactions may have to wait on each other. This could be avoided by using a Striped lock.

It's possible that allowing more concurrency may not be a good thing in that it could place more load on zookeeper. Could possibly make the stripe size small to allow a small amount of concurrency.

keith-turner avatar Jan 04 '19 22:01 keith-turner

I will take a look at it. Question, how do I properly test to see the performance benefits if any?

tynyttie avatar Sep 08 '20 12:09 tynyttie

@tynyttie I believe we now have a continuous ingest bulk load test in the accumulo-testing repository, that you may be able to adapt. Or, you may need to experiment and devise your own performance testing for something like this.

ctubbsii avatar Sep 08 '20 12:09 ctubbsii

I understand. Thank you.

tynyttie avatar Sep 08 '20 12:09 tynyttie

It appears that we do not want the striped lock to be used, and I believe @tynyttie is no longer actively participating in this community. I vote we simply close this ticket and unassign the issue for somebody else to pick up.

ivakegg avatar Apr 04 '22 16:04 ivakegg