You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by ji...@apache.org on 2016/06/07 23:35:16 UTC
incubator-geode git commit: GEODE-744: Incorrect use of
APP_FETCH_SIZE in GFSH
Repository: incubator-geode
Updated Branches:
refs/heads/develop 103a6bcdf -> ec466fc38
GEODE-744: Incorrect use of APP_FETCH_SIZE in GFSH
This closes #151
Project: http://git-wip-us.apache.org/repos/asf/incubator-geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/ec466fc3
Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/ec466fc3
Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/ec466fc3
Branch: refs/heads/develop
Commit: ec466fc38e8cc5fc65450362b8d313c35e6ea14f
Parents: 103a6bc
Author: Kevin J. Duling <kd...@pivotal.io>
Authored: Tue Jun 7 12:29:19 2016 -0700
Committer: Jinmei Liao <ji...@pivotal.io>
Committed: Tue Jun 7 16:34:30 2016 -0700
----------------------------------------------------------------------
.../internal/cli/functions/DataCommandFunction.java | 11 ++++-------
1 file changed, 4 insertions(+), 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/ec466fc3/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 ace24f2..8a4736b 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
@@ -77,6 +77,7 @@ import com.gemstone.gemfire.management.internal.cli.shell.Gfsh;
import com.gemstone.gemfire.management.internal.cli.util.JsonUtil;
import com.gemstone.gemfire.pdx.PdxInstance;
+import org.apache.commons.lang.StringUtils;
import org.apache.logging.log4j.Logger;
import org.json.JSONArray;
@@ -947,13 +948,9 @@ public class DataCommandFunction extends FunctionAdapter implements InternalEnt
}
private String addLimit(String query) {
- boolean containsLimitOrAggregate = query.contains(" limit")
- || query.contains(" LIMIT") || query.contains("count(*)");
- if(!containsLimitOrAggregate){
- String limitQuery = query + " limit " + getFetchSize();
- return limitQuery;
- }
- else return query;
+ if (StringUtils.containsIgnoreCase(query, " limit") || StringUtils.containsIgnoreCase(query, "count(*)"))
+ return query;
+ return query + " limit " + getFetchSize();
}
};