[
https://issues.apache.org/jira/browse/ACCUMULO-4788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16365802#comment-16365802
]
Keith Turner commented on ACCUMULO-4788:
----------------------------------------
In PR #385 the global lock was not addressed. The only change it made was to
make the operations done when the lock was held much faster.
> Improve Thrift Transport pool
> -----------------------------
>
> Key: ACCUMULO-4788
> URL: https://issues.apache.org/jira/browse/ACCUMULO-4788
> Project: Accumulo
> Issue Type: Improvement
> Reporter: Keith Turner
> Assignee: Keith Turner
> Priority: Major
> Labels: pull-request-available
> Fix For: 2.0.0
>
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> Accumulo has a pool of recently opened connections to tablet servers. When
> connecting to tablet servers, this pool is checked first. The pool is built
> around a map of list. There are two problems with this pool :
> * It has global lock around the map of list
> * When trying to find a connection it does a linear search for a non
> reserved connection (this is per tablet server)
> Could possibly move to a model of having a list of unreserved connections and
> a set of reserved connections per tablet server. Then to get a connection,
> could remove from the unreserved list and add to the reserved set. This
> would be a constant time operation.
> For the locking, could move to a model of using a concurrent map and locking
> per tserver instead of locking the entire map.
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
