You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by kl...@apache.org on 2016/06/16 18:16:09 UTC
[2/3] 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/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 <kd...@pivotal.io>
Authored: Fri Jun 10 10:03:44 2016 -0700
Committer: Kirk Lund <kl...@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();
}