You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ab...@apache.org on 2017/03/23 11:49:55 UTC

[08/46] lucene-solr:jira/solr-9959: SOLR-10270: Stop exporting _version_ during GROUP BY aggregations in map_reduce mode

SOLR-10270: Stop exporting _version_ during GROUP BY aggregations in map_reduce mode


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

Branch: refs/heads/jira/solr-9959
Commit: 22f91ba0cafeb04e1568cb8c61b5356ef7e91ade
Parents: 2bce98b
Author: Joel Bernstein <jb...@apache.org>
Authored: Mon Mar 13 12:50:17 2017 -0400
Committer: Joel Bernstein <jb...@apache.org>
Committed: Wed Mar 15 18:31:45 2017 -0400

----------------------------------------------------------------------
 .../core/src/java/org/apache/solr/handler/sql/SolrTable.java | 8 --------
 1 file changed, 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/22f91ba0/solr/core/src/java/org/apache/solr/handler/sql/SolrTable.java
----------------------------------------------------------------------
diff --git a/solr/core/src/java/org/apache/solr/handler/sql/SolrTable.java b/solr/core/src/java/org/apache/solr/handler/sql/SolrTable.java
index 9375bc0..37ed7d9 100644
--- a/solr/core/src/java/org/apache/solr/handler/sql/SolrTable.java
+++ b/solr/core/src/java/org/apache/solr/handler/sql/SolrTable.java
@@ -330,24 +330,16 @@ class SolrTable extends AbstractQueryableTable implements TranslatableTable {
 
   private String getFields(Set<String> fieldSet) {
     StringBuilder buf = new StringBuilder();
-    boolean appendVersion = true;
     for(String field : fieldSet) {
 
       if(buf.length() > 0) {
         buf.append(",");
       }
 
-      if(field.equals("_version_")) {
-        appendVersion = false;
-      }
 
       buf.append(field);
     }
 
-    if(appendVersion){
-      buf.append(",_version_");
-    }
-
     return buf.toString();
   }