You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by kr...@apache.org on 2016/10/11 20:27:46 UTC
lucene-solr:master: SOLR-8969: Fix static cloud check
Repository: lucene-solr
Updated Branches:
refs/heads/master b00f10916 -> 7660dde45
SOLR-8969: Fix static cloud check
Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/7660dde4
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/7660dde4
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/7660dde4
Branch: refs/heads/master
Commit: 7660dde458fd6f10cc7481d54a52ad7519b1a545
Parents: b00f109
Author: Kevin Risden <kr...@apache.org>
Authored: Tue Oct 11 14:48:25 2016 -0500
Committer: Kevin Risden <kr...@apache.org>
Committed: Tue Oct 11 15:27:30 2016 -0500
----------------------------------------------------------------------
.../src/java/org/apache/solr/handler/SQLHandler.java | 15 ++++++---------
1 file changed, 6 insertions(+), 9 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/7660dde4/solr/core/src/java/org/apache/solr/handler/SQLHandler.java
----------------------------------------------------------------------
diff --git a/solr/core/src/java/org/apache/solr/handler/SQLHandler.java b/solr/core/src/java/org/apache/solr/handler/SQLHandler.java
index 5fcf938..3074d9b 100644
--- a/solr/core/src/java/org/apache/solr/handler/SQLHandler.java
+++ b/solr/core/src/java/org/apache/solr/handler/SQLHandler.java
@@ -78,25 +78,22 @@ import com.facebook.presto.sql.parser.SqlParser;
public class SQLHandler extends RequestHandlerBase implements SolrCoreAware , PermissionNameProvider {
+ private static final Logger logger = LoggerFactory.getLogger(MethodHandles.lookup().lookupClass());
+
private static String defaultZkhost = null;
private static String defaultWorkerCollection = null;
- private static List<String> remove;
-
- static {
- remove = new ArrayList();
- remove.add("count(*)");
- }
- private static final Logger logger = LoggerFactory.getLogger(MethodHandles.lookup().lookupClass());
static final String sqlNonCloudErrorMsg = "/sql handler only works in Solr Cloud mode";
- public void inform(SolrCore core) {
+ private boolean isCloud = false;
+ public void inform(SolrCore core) {
CoreContainer coreContainer = core.getCoreDescriptor().getCoreContainer();
if(coreContainer.isZooKeeperAware()) {
defaultZkhost = core.getCoreDescriptor().getCoreContainer().getZkController().getZkServerAddress();
defaultWorkerCollection = core.getCoreDescriptor().getCollectionName();
+ isCloud = true;
}
}
@@ -121,7 +118,7 @@ public class SQLHandler extends RequestHandlerBase implements SolrCoreAware , Pe
try {
- if(workerZkhost == null) {
+ if(!isCloud) {
throw new IllegalStateException(sqlNonCloudErrorMsg);
}