You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by yo...@apache.org on 2016/10/25 01:19:24 UTC

lucene-solr:master: SOLR-9654: tests: specify descending count sort for streaming

Repository: lucene-solr
Updated Branches:
  refs/heads/master 9d692cde5 -> c9132ac66


SOLR-9654: tests: specify descending count sort for streaming


Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/c9132ac6
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/c9132ac6
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/c9132ac6

Branch: refs/heads/master
Commit: c9132ac66100ab46bea480397396105f8489b239
Parents: 9d692cd
Author: yonik <yo...@apache.org>
Authored: Mon Oct 24 21:18:51 2016 -0400
Committer: yonik <yo...@apache.org>
Committed: Mon Oct 24 21:19:06 2016 -0400

----------------------------------------------------------------------
 .../apache/solr/search/facet/TestJsonFacets.java    | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/c9132ac6/solr/core/src/test/org/apache/solr/search/facet/TestJsonFacets.java
----------------------------------------------------------------------
diff --git a/solr/core/src/test/org/apache/solr/search/facet/TestJsonFacets.java b/solr/core/src/test/org/apache/solr/search/facet/TestJsonFacets.java
index 0ec0be4..1c1a343 100644
--- a/solr/core/src/test/org/apache/solr/search/facet/TestJsonFacets.java
+++ b/solr/core/src/test/org/apache/solr/search/facet/TestJsonFacets.java
@@ -1151,10 +1151,10 @@ public class TestJsonFacets extends SolrTestCaseHS {
     if (!client.local()) {
       client.testJQ(params(p, "q", "*:*"
           , "json.facet", "{" +
-              "cat0:{type:terms, field:${cat_s}, limit:1, overrequest:0}" +
-              ",cat1:{type:terms, field:${cat_s}, limit:1, overrequest:1}" +
-              ",catDef:{type:terms, field:${cat_s}, limit:1, overrequest:-1}" +  // -1 is default overrequest
-              ",catBig:{type:terms, field:${cat_s}, offset:1, limit:2147483647, overrequest:2147483647}" +  // make sure overflows don't mess us up
+              "cat0:{type:terms, field:${cat_s}, sort:'count desc', limit:1, overrequest:0}" +
+              ",cat1:{type:terms, field:${cat_s}, sort:'count desc', limit:1, overrequest:1}" +
+              ",catDef:{type:terms, field:${cat_s}, sort:'count desc', limit:1, overrequest:-1}" +  // -1 is default overrequest
+              ",catBig:{type:terms, field:${cat_s}, sort:'count desc', offset:1, limit:2147483647, overrequest:2147483647}" +  // make sure overflows don't mess us up
               "}"
           )
           , "facets=={ count:6" +
@@ -1168,10 +1168,10 @@ public class TestJsonFacets extends SolrTestCaseHS {
       // In non-distrib mode, should still be able to specify overrequest, but it shouldn't matter.
       client.testJQ(params(p, "q", "*:*"
           , "json.facet", "{" +
-              "cat0:{type:terms, field:${cat_s}, limit:1, overrequest:0}" +
-              ",cat1:{type:terms, field:${cat_s}, limit:1, overrequest:1}" +
-              ",catDef:{type:terms, field:${cat_s}, limit:1, overrequest:-1}" +  // -1 is default overrequest
-              ",catBig:{type:terms, field:${cat_s}, offset:1, limit:2147483647, overrequest:2147483647}" +  // make sure overflows don't mess us up
+              "cat0:{type:terms, field:${cat_s}, sort:'count desc', limit:1, overrequest:0}" +
+              ",cat1:{type:terms, field:${cat_s}, sort:'count desc', limit:1, overrequest:1}" +
+              ",catDef:{type:terms, field:${cat_s}, sort:'count desc', limit:1, overrequest:-1}" +  // -1 is default overrequest
+              ",catBig:{type:terms, field:${cat_s}, sort:'count desc', offset:1, limit:2147483647, overrequest:2147483647}" +  // make sure overflows don't mess us up
               "}"
           )
           , "facets=={ count:6" +