> On Sept. 3, 2014, 9:39 p.m., kturner wrote: > > should anything be done in the shell relating to durability? could > > possibly allow inserts done via the shell to be done w/ differing > > durability. This may be useful is script wants to do a lot of inserts via > > the shell. > > Josh Elser wrote: > I could see that as a follow-on (if at all). Anyone wanting to script > some simple inserts should probably be using the proxy (I assume the perf is > much better) which should get these fixes automatically, right?
Right, using the proxy would be better. Each insert from the shell still makes a round trip RPC. So it would still not be ideal. Batch writing from the proxy would be better. - kturner ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25117/#review52248 ----------------------------------------------------------- On Sept. 3, 2014, 9:42 p.m., Eric Newton wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25117/ > ----------------------------------------------------------- > > (Updated Sept. 3, 2014, 9:42 p.m.) > > > Review request for accumulo, Josh Elser, kturner, and John Vines. > > > Bugs: ACCUMULO-1957 > https://issues.apache.org/jira/browse/ACCUMULO-1957 > > > Repository: accumulo > > > Description > ------- > > Allow tables and update sessions to be configured with different levels of > durability (WAL sync'ing). > > > Diffs > ----- > > core/src/main/java/org/apache/accumulo/core/client/BatchWriterConfig.java > e2ec22e > > core/src/main/java/org/apache/accumulo/core/client/ConditionalWriterConfig.java > 7671c35 > core/src/main/java/org/apache/accumulo/core/client/Durability.java > PRE-CREATION > > core/src/main/java/org/apache/accumulo/core/client/impl/ConditionalWriterImpl.java > f5e6dd2 > > core/src/main/java/org/apache/accumulo/core/client/impl/TabletServerBatchWriter.java > f2dd980 > core/src/main/java/org/apache/accumulo/core/client/impl/Writer.java bf226eb > core/src/main/java/org/apache/accumulo/core/conf/Property.java 9837867 > core/src/main/java/org/apache/accumulo/core/conf/PropertyType.java f39a8bd > > core/src/main/java/org/apache/accumulo/core/master/thrift/MasterClientService.java > 9c850a8 > > core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/TDurability.java > PRE-CREATION > > core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/TabletClientService.java > 2ba7674 > core/src/main/thrift/tabletserver.thrift 25e0b10 > > examples/simple/src/main/java/org/apache/accumulo/examples/simple/client/ReadWriteExample.java > a7b288d > server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java > 9b952ba > server/tserver/src/main/java/org/apache/accumulo/tserver/Mutations.java > PRE-CREATION > > server/tserver/src/main/java/org/apache/accumulo/tserver/TabletMutations.java > e814f0e > server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java > 57e3dee > server/tserver/src/main/java/org/apache/accumulo/tserver/log/DfsLogger.java > c01e54a > > server/tserver/src/main/java/org/apache/accumulo/tserver/log/TabletServerLogger.java > 26e6891 > > server/tserver/src/main/java/org/apache/accumulo/tserver/session/ConditionalSession.java > 26668f6 > > server/tserver/src/main/java/org/apache/accumulo/tserver/session/UpdateSession.java > bc04a85 > > server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CommitSession.java > 6402797 > server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/Tablet.java > 37950fc > > server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/TabletCommitter.java > a5d197c > test/src/main/java/org/apache/accumulo/test/WrongTabletTest.java aeba2e0 > > test/src/main/java/org/apache/accumulo/test/performance/thrift/NullTserver.java > 6c34172 > test/src/test/java/org/apache/accumulo/test/functional/DurabilityIT.java > PRE-CREATION > > test/src/test/java/org/apache/accumulo/test/functional/SessionDurabilityIT.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/25117/diff/ > > > Testing > ------- > > Added DurabilityIT. > > > Thanks, > > Eric Newton > >
