Updated Branches:
  refs/heads/master c16539486 -> b4ce6f88e

ACCUMULO-1523 applying Bill Havanki's patch


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/b4ce6f88
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/b4ce6f88
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/b4ce6f88

Branch: refs/heads/master
Commit: b4ce6f88eb5db3c9a58e9a2d298447cdd2c44287
Parents: c165394
Author: Eric Newton <[email protected]>
Authored: Thu Sep 19 14:08:04 2013 -0400
Committer: Eric Newton <[email protected]>
Committed: Thu Sep 19 14:08:04 2013 -0400

----------------------------------------------------------------------
 core/src/main/java/org/apache/accumulo/core/conf/Property.java    | 3 ++-
 .../org/apache/accumulo/server/tabletserver/TabletServer.java     | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/b4ce6f88/core/src/main/java/org/apache/accumulo/core/conf/Property.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/accumulo/core/conf/Property.java 
b/core/src/main/java/org/apache/accumulo/core/conf/Property.java
index e58e771..b6fbdd2 100644
--- a/core/src/main/java/org/apache/accumulo/core/conf/Property.java
+++ b/core/src/main/java/org/apache/accumulo/core/conf/Property.java
@@ -109,7 +109,7 @@ public enum Property {
   GENERAL_KERBEROS_KEYTAB("general.kerberos.keytab", "", PropertyType.PATH, 
"Path to the kerberos keytab to use. Leave blank if not using kerberoized 
hdfs"),
   GENERAL_KERBEROS_PRINCIPAL("general.kerberos.principal", "", 
PropertyType.STRING, "Name of the kerberos principal to use. _HOST will 
automatically be "
       + "replaced by the machines hostname in the hostname portion of the 
principal. Leave blank if not using kerberoized hdfs"),
-  GENERAL_MAX_MESSAGE_SIZE("tserver.server.message.size.max", "1G", 
PropertyType.MEMORY, "The maximum size of a message that can be sent to a 
tablet server."),
+  GENERAL_MAX_MESSAGE_SIZE("general.server.message.size.max", "1G", 
PropertyType.MEMORY, "The maximum size of a message that can be sent to a 
server."),
   GENERAL_VOLUME_CHOOSER("general.volume.chooser", 
"org.apache.accumulo.server.fs.RandomVolumeChooser", PropertyType.CLASSNAME, 
"The class that will be used to select which volume will be used to create new 
files."),
 
   // properties that are specific to master server behavior
@@ -200,6 +200,7 @@ public enum Property {
   TSERV_BULK_TIMEOUT("tserver.bulk.timeout", "5m", PropertyType.TIMEDURATION, 
"The time to wait for a tablet server to process a bulk import request."),
   TSERV_MINTHREADS("tserver.server.threads.minimum", "20", PropertyType.COUNT, 
"The minimum number of threads to use to handle incoming requests."),
   TSERV_THREADCHECK("tserver.server.threadcheck.time", "1s", 
PropertyType.TIMEDURATION, "The time between adjustments of the server thread 
pool."),
+  TSERV_MAX_MESSAGE_SIZE("tserver.server.message.size.max", "1G", 
PropertyType.MEMORY, "The maximum size of a message that can be sent to a 
tablet server."),
   TSERV_HOLD_TIME_SUICIDE("tserver.hold.time.max", "5m", 
PropertyType.TIMEDURATION,
       "The maximum time for a tablet server to be in the \"memory full\" 
state.  If the tablet server cannot write out memory"
           + " in this much time, it will assume there is some failure local to 
its node, and quit.  A value of zero is equivalent to forever."),

http://git-wip-us.apache.org/repos/asf/accumulo/blob/b4ce6f88/server/src/main/java/org/apache/accumulo/server/tabletserver/TabletServer.java
----------------------------------------------------------------------
diff --git 
a/server/src/main/java/org/apache/accumulo/server/tabletserver/TabletServer.java
 
b/server/src/main/java/org/apache/accumulo/server/tabletserver/TabletServer.java
index 7ee0a45..8959d4f 100644
--- 
a/server/src/main/java/org/apache/accumulo/server/tabletserver/TabletServer.java
+++ 
b/server/src/main/java/org/apache/accumulo/server/tabletserver/TabletServer.java
@@ -2936,8 +2936,9 @@ public class TabletServer extends AbstractMetricsImpl 
implements org.apache.accu
   }
   
   private InetSocketAddress startServer(AccumuloConfiguration conf, String 
address, Property portHint, TProcessor processor, String threadName) throws 
UnknownHostException {
+    Property maxMessageSizeProperty = 
(conf.get(Property.TSERV_MAX_MESSAGE_SIZE) != null ? 
Property.TSERV_MAX_MESSAGE_SIZE : Property.GENERAL_MAX_MESSAGE_SIZE);
     ServerAddress sp = TServerUtils.startServer(conf, address, portHint, 
processor, this.getClass().getSimpleName(), threadName, 
Property.TSERV_PORTSEARCH,
-        Property.TSERV_MINTHREADS, Property.TSERV_THREADCHECK, 
Property.GENERAL_MAX_MESSAGE_SIZE);
+        Property.TSERV_MINTHREADS, Property.TSERV_THREADCHECK, 
maxMessageSizeProperty);
     this.server = sp.server;
     return sp.address;
   }

Reply via email to