-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17426/
-----------------------------------------------------------
(Updated Feb. 4, 2014, 1:43 p.m.)
Review request for accumulo.
Changes
-------
Adjusted comment in TabletServerResourceManager to be not "unexpected". Other
changes in this diff from the last one are from Findbugs-based improvements
introduced since the last diff under a different ticket.
Bugs: ACCUMULO-1948
https://issues.apache.org/jira/browse/ACCUMULO-1948
Repository: accumulo
Description
-------
Change in the construction of the cross-referencing between Tablet and
TabletResourceManager to avoid the Tablet constructor leaking this.
Diffs (updated)
-----
server/tserver/pom.xml b627de092faa2b5b4dbc5f8b9e4fb7af696ca436
server/tserver/src/main/java/org/apache/accumulo/tserver/Tablet.java
7ce270b361f1df212fa3bd3f46e46c3b2b080df3
server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java
080cc203e9230c62ca913fa9bf80d671bdd2e303
server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServerResourceManager.java
e95843775c85ecf90a2b4c7afce91094381da450
server/tserver/src/test/java/org/apache/accumulo/tserver/TabletResourceManagerTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/17426/diff/
Testing
-------
Added unit test. Tested on pseudo-cluster with shell activity, randomwalk
MultiTable test.
Thanks,
Bill Havanki
