accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Havanki" <bhava...@clouderagovt.com>
Subject Re: Review Request 17426: ACCUMULO-1948 Tablet constructor no longer leaking this
Date Tue, 28 Jan 2014 12:47:44 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17426/
-----------------------------------------------------------

(Updated Jan. 28, 2014, 7:47 a.m.)


Review request for accumulo.


Changes
-------

Simplified API - table configuration now retrieved from tablet itself.


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 40dfcf0fa1715fbb0ff69e78bdf8954146a36b20

  server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java 5c1f6ce5f95523ef16005902c99a8f974e9e2baf

  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. Real cluster testing TBD.


Thanks,

Bill Havanki


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message