> On June 17, 2014, 3:55 a.m., Mike Drob wrote: > > core/src/main/java/org/apache/accumulo/core/client/impl/Writer.java, line > > 165 > > <https://reviews.apache.org/r/22658/diff/1/?file=610893#file610893line165> > > > > Lifecycle management: what happens if a user forgets to call commit? > > Maybe JavaDoc that it is important.
s/user/server/ ? > On June 17, 2014, 3:55 a.m., Mike Drob wrote: > > server/base/src/main/java/org/apache/accumulo/server/util/MetadataTableUtil.java, > > lines 139-150 > > <https://reviews.apache.org/r/22658/diff/1/?file=610894#file610894line139> > > > > Infinite retry makes me uneasy. Especially with no back-off in between. This is pretty typical. If the master is dead, we're already dead in the water. We want to keep trying until we get a success. - Josh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22658/#review45893 ----------------------------------------------------------- On June 16, 2014, 9:59 p.m., Jonathan Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/22658/ > ----------------------------------------------------------- > > (Updated June 16, 2014, 9:59 p.m.) > > > Review request for accumulo. > > > Bugs: ACCUMULO-2889 > https://issues.apache.org/jira/browse/ACCUMULO-2889 > > > Repository: accumulo > > > Description > ------- > > Added additional methods to the Writer class to handle batching. > > Potential risks are that we're now holding onto the locks for a bit longer > than we used to. All tablets present in a batch will have their logLock's > locked until the batch is complete. > > > Diffs > ----- > > core/src/main/java/org/apache/accumulo/core/client/impl/Writer.java d6762e7 > > server/base/src/main/java/org/apache/accumulo/server/util/MetadataTableUtil.java > 374017d > server/tserver/src/main/java/org/apache/accumulo/tserver/Tablet.java > 36b2289 > server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java > e2510d7 > > server/tserver/src/main/java/org/apache/accumulo/tserver/log/TabletServerLogger.java > d25ee75 > test/src/test/java/org/apache/accumulo/test/BatchMetadataUpdatesIT.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/22658/diff/ > > > Testing > ------- > > Added a new IT. > > We insert a few entries to tablets and ensure that the relevant entries > appear in the WAL and Metadata Table. > One test case that isn't included yet is verifying that root + metadata table > entries are entered correctly. > > > Thanks, > > Jonathan Park > >
