Repository: incubator-impala
Updated Branches:
  refs/heads/master ad0c6e749 -> 304edb28c


IMPALA-3937: Deprecate --be_service_threads

FLAGS_be_service_threads does nothing, and can be removed. Backend
Thrift servers do not use a fix-sized thread pool, instead using one
thread-per-connection.

Change-Id: I10e48014f24eebd22251bac4734bc3c90dee47c0
Reviewed-on: http://gerrit.cloudera.org:8080/7483
Reviewed-by: Matthew Jacobs <[email protected]>
Tested-by: Impala Public Jenkins


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

Branch: refs/heads/master
Commit: ed7324431d16a37a279d730a036197fc9019c3ce
Parents: ad0c6e7
Author: Henry Robinson <[email protected]>
Authored: Fri Jul 21 16:20:55 2017 -0700
Committer: Impala Public Jenkins <[email protected]>
Committed: Sat Jul 22 03:44:01 2017 +0000

----------------------------------------------------------------------
 be/src/service/impala-server.cc | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/ed732443/be/src/service/impala-server.cc
----------------------------------------------------------------------
diff --git a/be/src/service/impala-server.cc b/be/src/service/impala-server.cc
index a26f0ab..c3d22fe 100644
--- a/be/src/service/impala-server.cc
+++ b/be/src/service/impala-server.cc
@@ -114,8 +114,8 @@ DEFINE_int32(hs2_port, 21050, "port on which HiveServer2 
client requests are ser
 
 DEFINE_int32(fe_service_threads, 64,
     "number of threads available to serve client requests");
-DEFINE_int32(be_service_threads, 64,
-    "(Advanced) number of threads available to serve backend execution 
requests");
+DEFINE_int32_hidden(be_service_threads, 64,
+    "Deprecated, no longer has any effect. Will be removed in Impala 3.0.");
 DEFINE_string(default_query_options, "", "key=value pair of default query 
options for"
     " impalad, separated by ','");
 DEFINE_int32(query_log_size, 25, "Number of queries to retain in the query 
log. If -1, "
@@ -1934,7 +1934,7 @@ Status CreateImpalaServer(ExecEnv* exec_env, int 
beeswax_port, int hs2_port, int
     be_processor->setEventHandler(event_handler);
 
     *be_server = new ThriftServer("backend", be_processor, be_port, nullptr,
-        exec_env->metrics(), FLAGS_be_service_threads);
+        exec_env->metrics());
     if (EnableInternalSslConnections()) {
       LOG(INFO) << "Enabling SSL for backend";
       RETURN_IF_ERROR((*be_server)->EnableSsl(FLAGS_ssl_server_certificate,

Reply via email to