You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-dev@lucene.apache.org by "Sean Fitzgerald (JIRA)" <ji...@apache.org> on 2009/09/25 23:49:16 UTC

[jira] Created: (SOLR-1464) CommonsHttpSolrServer does not conform to bean conventions

CommonsHttpSolrServer does not conform to bean conventions
----------------------------------------------------------

                 Key: SOLR-1464
                 URL: https://issues.apache.org/jira/browse/SOLR-1464
             Project: Solr
          Issue Type: Bug
          Components: clients - java
    Affects Versions: 1.3
            Reporter: Sean Fitzgerald
         Attachments: CommonsHttpSolrServer.java-BEAN.patch

Several class variables (baseURL, allowCompression, maxRetries, etc) have neither getters nor setters. By creating getters and setters for these properties, we can allow other developers to extend CommonsHttpSolrServer with additional functionality. It is also then necessary to use these methods internally, as opposed to referencing the class variables directly.



For example, by extending a method like 
public String getBaseURL()
One could attach a host monitoring or home-brewed DNS resolution service to intercept, thus replicating the functionality of LBHttpSolrServer with very little of the code.

Attached is a basic patch (generated using eclipse Source tools), as a minimal set of changes. I have not changes the general coding style of the file, though that would be preferable. I am open to suggestion on whether these methods should be public (as in the attached patch), or protected.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SOLR-1464) CommonsHttpSolrServer does not conform to bean conventions

Posted by "Sean Fitzgerald (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SOLR-1464?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sean Fitzgerald updated SOLR-1464:
----------------------------------

    Attachment: CommonsHttpSolrServer.java-BEAN.patch

Generated bean methods for CommonsHttpSolrServer.java

> CommonsHttpSolrServer does not conform to bean conventions
> ----------------------------------------------------------
>
>                 Key: SOLR-1464
>                 URL: https://issues.apache.org/jira/browse/SOLR-1464
>             Project: Solr
>          Issue Type: Bug
>          Components: clients - java
>    Affects Versions: 1.3
>            Reporter: Sean Fitzgerald
>         Attachments: CommonsHttpSolrServer.java-BEAN.patch
>
>
> Several class variables (baseURL, allowCompression, maxRetries, etc) have neither getters nor setters. By creating getters and setters for these properties, we can allow other developers to extend CommonsHttpSolrServer with additional functionality. It is also then necessary to use these methods internally, as opposed to referencing the class variables directly.
> For example, by extending a method like 
> public String getBaseURL()
> One could attach a host monitoring or home-brewed DNS resolution service to intercept, thus replicating the functionality of LBHttpSolrServer with very little of the code.
> Attached is a basic patch (generated using eclipse Source tools), as a minimal set of changes. I have not changes the general coding style of the file, though that would be preferable. I am open to suggestion on whether these methods should be public (as in the attached patch), or protected.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.