You are viewing a plain text version of this content. The canonical link for it is here.
Posted to droids-commits@incubator.apache.org by to...@apache.org on 2013/04/03 08:05:06 UTC

svn commit: r1463869 - /incubator/droids/branches/0.2.x-cleanup/droids-solr/src/main/java/org/apache/droids/solr/SolrHandler.java

Author: tobr
Date: Wed Apr  3 08:05:06 2013
New Revision: 1463869

URL: http://svn.apache.org/r1463869
Log:
removed setters for members that should be set in the constructor

Modified:
    incubator/droids/branches/0.2.x-cleanup/droids-solr/src/main/java/org/apache/droids/solr/SolrHandler.java

Modified: incubator/droids/branches/0.2.x-cleanup/droids-solr/src/main/java/org/apache/droids/solr/SolrHandler.java
URL: http://svn.apache.org/viewvc/incubator/droids/branches/0.2.x-cleanup/droids-solr/src/main/java/org/apache/droids/solr/SolrHandler.java?rev=1463869&r1=1463868&r2=1463869&view=diff
==============================================================================
--- incubator/droids/branches/0.2.x-cleanup/droids-solr/src/main/java/org/apache/droids/solr/SolrHandler.java (original)
+++ incubator/droids/branches/0.2.x-cleanup/droids-solr/src/main/java/org/apache/droids/solr/SolrHandler.java Wed Apr  3 08:05:06 2013
@@ -33,7 +33,7 @@ import java.io.IOException;
 public abstract class SolrHandler implements Handler {
 
     private SolrServer solrServer;
-    protected String solrUrl;
+    private String solrUrl;
 
     protected final static Logger logger = LoggerFactory.getLogger(SolrHandler.class);
 
@@ -45,7 +45,8 @@ public abstract class SolrHandler implem
      * @param solrUrl the url to the solr server
      */
     public SolrHandler(String solrUrl) {
-        this.setSolrUrl(solrUrl);
+        this.solrUrl = solrUrl;
+        this.solrServer = createHttpSolrServer();
     }
 
     /**
@@ -112,15 +113,6 @@ public abstract class SolrHandler implem
     }
 
     /**
-     * Set the solr server.
-     *
-     * @param solrServer the solr server
-     */
-    public void setSolrServer(SolrServer solrServer) {
-        this.solrServer = solrServer;
-    }
-
-    /**
      * Get the solr server.
      *
      * @return the solr server instance
@@ -130,16 +122,6 @@ public abstract class SolrHandler implem
     }
 
     /**
-     * Set the url of the solr server
-     *
-     * @param solrUrl the url of the solr server
-     */
-    public void setSolrUrl(String solrUrl) {
-        this.solrUrl = solrUrl;
-        this.solrServer = createHttpSolrServer();
-    }
-
-    /**
      * Get the url of the solr server.
      * Can also be null, if the solr server has no url.
      *