You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by kt...@apache.org on 2012/09/10 17:25:54 UTC
svn commit: r1382940 -
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/TabletServerBatchWriter.java
Author: kturner
Date: Mon Sep 10 15:25:54 2012
New Revision: 1382940
URL: http://svn.apache.org/viewvc?rev=1382940&view=rev
Log:
ACCUMULO-760 Applied patch from Christopher Tubbs to remove sync for BatchWriter addMutation() that takes an iterator
Modified:
accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/TabletServerBatchWriter.java
Modified: accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/TabletServerBatchWriter.java
URL: http://svn.apache.org/viewvc/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/TabletServerBatchWriter.java?rev=1382940&r1=1382939&r2=1382940&view=diff
==============================================================================
--- accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/TabletServerBatchWriter.java (original)
+++ accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/TabletServerBatchWriter.java Mon Sep 10 15:25:54 2012
@@ -72,7 +72,6 @@ import org.apache.thrift.TServiceClient;
import org.apache.thrift.transport.TTransport;
import org.apache.thrift.transport.TTransportException;
-
/*
* Differences from previous TabletServerBatchWriter
* + As background threads finish sending mutations to tablet servers they decrement memory usage
@@ -242,7 +241,7 @@ public class TabletServerBatchWriter {
}
}
- public synchronized void addMutation(String table, Iterator<Mutation> iterator) throws MutationsRejectedException {
+ public void addMutation(String table, Iterator<Mutation> iterator) throws MutationsRejectedException {
while (iterator.hasNext()) {
addMutation(table, iterator.next());
}