You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lens.apache.org by am...@apache.org on 2014/12/23 13:12:25 UTC

incubator-lens git commit: LENS-18 : Adds queryHandle counter while doing query listing (Raju Bairishetti via amareshwari)

Repository: incubator-lens
Updated Branches:
  refs/heads/master f6f675e2a -> f18e60097


LENS-18 : Adds queryHandle counter while doing query listing (Raju Bairishetti via amareshwari)


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

Branch: refs/heads/master
Commit: f18e600973075a3ad472cf8fde8578d5bbff1c14
Parents: f6f675e
Author: Amareshwari Sriramdasu <am...@inmobi.com>
Authored: Tue Dec 23 17:42:16 2014 +0530
Committer: Amareshwari Sriramdasu <am...@inmobi.com>
Committed: Tue Dec 23 17:42:16 2014 +0530

----------------------------------------------------------------------
 .../java/org/apache/lens/cli/commands/LensQueryCommands.java     | 3 ++-
 .../src/test/java/org/apache/lens/cli/TestLensQueryCommands.java | 4 ++++
 2 files changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-lens/blob/f18e6009/lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java
----------------------------------------------------------------------
diff --git a/lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java b/lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java
index 32cdf28..81cc5cf 100644
--- a/lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java
+++ b/lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java
@@ -191,7 +191,8 @@ public class LensQueryCommands extends BaseLensCommand implements CommandMarker
     }
     List<QueryHandle> handles = getClient().getQueries(state, queryName, user, fromDate, toDate);
     if (handles != null && !handles.isEmpty()) {
-      return Joiner.on("\n").skipNulls().join(handles);
+      return Joiner.on("\n").skipNulls().join(handles).concat("\n").concat("Total number of queries: "
+          + handles.size());
     } else {
       return "No queries";
     }

http://git-wip-us.apache.org/repos/asf/incubator-lens/blob/f18e6009/lens-cli/src/test/java/org/apache/lens/cli/TestLensQueryCommands.java
----------------------------------------------------------------------
diff --git a/lens-cli/src/test/java/org/apache/lens/cli/TestLensQueryCommands.java b/lens-cli/src/test/java/org/apache/lens/cli/TestLensQueryCommands.java
index de29094..2f9e94d 100644
--- a/lens-cli/src/test/java/org/apache/lens/cli/TestLensQueryCommands.java
+++ b/lens-cli/src/test/java/org/apache/lens/cli/TestLensQueryCommands.java
@@ -184,6 +184,10 @@ public class TestLensQueryCommands extends LensCliApplicationTest {
     String result = qCom.getAllQueries("", "testQuery1", user, -1, Long.MAX_VALUE);
     // this is because previous query has run two query handle will be there
     Assert.assertTrue(result.contains(qh), result);
+    Assert.assertTrue(result.contains("Total number of queries"));
+    String [] resultSplits = result.split("\n");
+    // assert on the number of queries
+    Assert.assertEquals(String.valueOf(resultSplits.length -1), resultSplits[resultSplits.length-1].split(": ")[1]);
 
     // Check that query name searching is 'ilike'
     String result2 = qCom.getAllQueries("", "query", "all", -1, Long.MAX_VALUE);