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 <[email protected]>
Authored: Thu May 21 01:05:58 2015 -0700
Committer: Madhan Neethiraj <[email protected]>
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);

Reply via email to