geode-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kl...@apache.org
Subject [2/3] incubator-geode git commit: GEODE-1521 - APP_FETCH_SIZE in GFSH should not be applied to COUNT queries
Date Thu, 16 Jun 2016 18:16:09 GMT
GEODE-1521 - APP_FETCH_SIZE in GFSH should not be applied to COUNT queries

Change to search for " count(" instead of "count(*)" because 'count' can accept more options
than simply a star.

* This closes #155 [klund@apache.org]


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

Branch: refs/heads/develop
Commit: b3955c16909c9d780bfc1a539ed428cf40b7bd2c
Parents: 1c5ba14
Author: Kevin J. Duling <kduling@pivotal.io>
Authored: Fri Jun 10 10:03:44 2016 -0700
Committer: Kirk Lund <klund@pivotal.io>
Committed: Thu Jun 16 11:15:59 2016 -0700

----------------------------------------------------------------------
 .../management/internal/cli/functions/DataCommandFunction.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/b3955c16/geode-core/src/main/java/com/gemstone/gemfire/management/internal/cli/functions/DataCommandFunction.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/com/gemstone/gemfire/management/internal/cli/functions/DataCommandFunction.java
b/geode-core/src/main/java/com/gemstone/gemfire/management/internal/cli/functions/DataCommandFunction.java
index 8a4736b..48cc83a 100644
--- a/geode-core/src/main/java/com/gemstone/gemfire/management/internal/cli/functions/DataCommandFunction.java
+++ b/geode-core/src/main/java/com/gemstone/gemfire/management/internal/cli/functions/DataCommandFunction.java
@@ -948,7 +948,7 @@ public class DataCommandFunction extends FunctionAdapter implements  InternalEnt
     }
 
     private String addLimit(String query) {
-      if (StringUtils.containsIgnoreCase(query, " limit") || StringUtils.containsIgnoreCase(query,
"count(*)"))
+      if (StringUtils.containsIgnoreCase(query, " limit") || StringUtils.containsIgnoreCase(query,
" count("))
         return query;
       return query + " limit " + getFetchSize();
     }


Mime
View raw message