kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From danburk...@apache.org
Subject [2/2] kudu git commit: [java client] removes duplicate checks at KuduScanToken
Date Mon, 15 May 2017 22:16:59 GMT
[java client] removes duplicate checks at KuduScanToken

KUDU-579 added fault tolerant scanner support at java
client. However,it missed TODO for KUDU-1040 and results
in duplicate code at KuduScanToken. This patch removes it.

Change-Id: I7e036d5fe333009a74294d27c54fe846cdc404e9
Reviewed-on: http://gerrit.cloudera.org:8080/6882
Tested-by: Kudu Jenkins
Reviewed-by: Todd Lipcon <todd@apache.org>


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

Branch: refs/heads/master
Commit: b28a59266ca842a5e6e6ced44af4160fab4b528f
Parents: e270589
Author: hahao <hao.hao@cloudera.com>
Authored: Fri May 12 16:30:26 2017 -0700
Committer: Todd Lipcon <todd@apache.org>
Committed: Mon May 15 21:38:10 2017 +0000

----------------------------------------------------------------------
 .../src/main/java/org/apache/kudu/client/KuduScanToken.java      | 4 ----
 1 file changed, 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/b28a5926/java/kudu-client/src/main/java/org/apache/kudu/client/KuduScanToken.java
----------------------------------------------------------------------
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/KuduScanToken.java b/java/kudu-client/src/main/java/org/apache/kudu/client/KuduScanToken.java
index 361df0e..6dbe286 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/KuduScanToken.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/KuduScanToken.java
@@ -195,10 +195,6 @@ public class KuduScanToken implements Comparable<KuduScanToken>
{
       builder.limit(message.getLimit());
     }
 
-    if (message.hasFaultTolerant()) {
-      // TODO(KUDU-1040)
-    }
-
     if (message.hasReadMode()) {
       switch (message.getReadMode()) {
         case READ_AT_SNAPSHOT: {


Mime
View raw message