You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by ma...@apache.org on 2015/05/22 00:56:16 UTC

[2/6] incubator-ranger git commit: RANGER-500 - Check for url property also

RANGER-500 - Check for url property also


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

Branch: refs/heads/tag-policy
Commit: 6568faabbdb2a593ad276c8a9db2a0523195ed6e
Parents: b8d5bb2
Author: Don Bosco Durai <bo...@apache.org>
Authored: Thu May 21 01:05:58 2015 -0700
Committer: Madhan Neethiraj <ma...@apache.org>
Committed: Thu May 21 15:50:16 2015 -0700

----------------------------------------------------------------------
 .../src/main/java/org/apache/ranger/solr/SolrMgr.java | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-ranger/blob/6568faab/security-admin/src/main/java/org/apache/ranger/solr/SolrMgr.java
----------------------------------------------------------------------
diff --git a/security-admin/src/main/java/org/apache/ranger/solr/SolrMgr.java b/security-admin/src/main/java/org/apache/ranger/solr/SolrMgr.java
index ee52992..2a57fa1 100644
--- a/security-admin/src/main/java/org/apache/ranger/solr/SolrMgr.java
+++ b/security-admin/src/main/java/org/apache/ranger/solr/SolrMgr.java
@@ -48,6 +48,8 @@ public class SolrMgr {
 	Date lastConnectTime = null;
 	volatile boolean initDone = false;
 
+	final static String SOLR_URLS_PROP = "ranger.audit.solr.urls";
+
 	public SolrMgr() {
 
 	}
@@ -58,16 +60,22 @@ public class SolrMgr {
 				if (!initDone) {
 					if (rangerBizUtil.getAuditDBType().equalsIgnoreCase("solr")) {
 						String solrURL = PropertiesUtil
-								.getProperty("ranger.audit.solr.urls");
+								.getProperty(SOLR_URLS_PROP);
 
 						if (solrURL == null) {
-							//Let's try older property name
+							// Try with url
+							solrURL = PropertiesUtil
+									.getProperty("ranger.audit.solr.url");
+						}
+						if (solrURL == null) {
+							// Let's try older property name
 							solrURL = PropertiesUtil
 									.getProperty("ranger.solr.url");
 						}
 						if (solrURL == null || solrURL.isEmpty()
 								|| solrURL.equalsIgnoreCase("none")) {
-							logger.fatal("Solr URL for Audit is empty");
+							logger.fatal("Solr URL for Audit is empty. Please set property "
+									+ SOLR_URLS_PROP);
 						} else {
 							try {
 								solrClient = new HttpSolrClient(solrURL);