impala-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From mjac...@apache.org
Subject [1/9] incubator-impala git commit: IMPALA-4937: Remove unused kudu scanner keep alive variable
Date Fri, 17 Feb 2017 23:17:04 GMT
Repository: incubator-impala
Updated Branches:
  refs/heads/master 26eaa2660 -> 815c76f9c


IMPALA-4937: Remove unused kudu scanner keep alive variable

gflag kudu_scanner_keep_alive_period_us is defined but not
used. The same functionality is provided by
kudu_scanner_keep_alive_period_sec (in seconds), which works.
The kudu_scanner_keep_alive_period_us flag is removed by
this patch.

Change-Id: I99f56f93f1ff8543cbe415542464a053b24968e2
Reviewed-on: http://gerrit.cloudera.org:8080/6021
Reviewed-by: Sailesh Mukil <sailesh@cloudera.com>
Reviewed-by: Alex Behm <alex.behm@cloudera.com>
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/aee5457a
Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/aee5457a
Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/aee5457a

Branch: refs/heads/master
Commit: aee5457a55d78203576b85440bc94fd3f6469fe0
Parents: 26eaa26
Author: Matthew Jacobs <mj@cloudera.com>
Authored: Wed Feb 15 11:01:29 2017 -0800
Committer: Impala Public Jenkins <impala-public-jenkins@gerrit.cloudera.org>
Committed: Thu Feb 16 22:25:13 2017 +0000

----------------------------------------------------------------------
 be/src/exec/kudu-scan-node.cc | 3 ---
 1 file changed, 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/aee5457a/be/src/exec/kudu-scan-node.cc
----------------------------------------------------------------------
diff --git a/be/src/exec/kudu-scan-node.cc b/be/src/exec/kudu-scan-node.cc
index 630f5b7..a238e3c 100644
--- a/be/src/exec/kudu-scan-node.cc
+++ b/be/src/exec/kudu-scan-node.cc
@@ -44,9 +44,6 @@
 
 DEFINE_int32(kudu_max_row_batches, 0, "The maximum size of the row batch queue, "
     " for Kudu scanners.");
-DEFINE_int32(kudu_scanner_keep_alive_period_us, 15 * 1000L * 1000L,
-    "The period at which Kudu Scanners should send keep-alive requests to the tablet "
-    "server to ensure that scanners do not time out.");
 
 using boost::algorithm::to_lower_copy;
 using kudu::client::KuduClient;


Mime
View raw message