You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by ud...@apache.org on 2016/06/16 22:07:27 UTC

[13/17] incubator-geode git commit: GEODE-1521 - APP_FETCH_SIZE in GFSH should not be applied to COUNT queries

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/f5093deb
Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/f5093deb
Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/f5093deb

Branch: refs/heads/feature/GEODE-420
Commit: f5093debbed715615ff773a14e1ed3d4d690ccc5
Parents: 921bec1
Author: Kevin J. Duling <kd...@pivotal.io>
Authored: Fri Jun 10 10:03:44 2016 -0700
Committer: Udo Kohlmeyer <uk...@pivotal.io>
Committed: Fri Jun 17 08:05:58 2016 +1000

----------------------------------------------------------------------
 .../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/f5093deb/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();
     }