lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alan Woodward (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (SOLR-7151) SolrClient.query() methods should throw IOException
Date Tue, 24 Feb 2015 11:00:06 GMT

     [ https://issues.apache.org/jira/browse/SOLR-7151?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Alan Woodward updated SOLR-7151:
--------------------------------
    Attachment: SOLR-7151.patch

Patch, building on top of the patch on SOLR-7145.

> SolrClient.query() methods should throw IOException
> ---------------------------------------------------
>
>                 Key: SOLR-7151
>                 URL: https://issues.apache.org/jira/browse/SOLR-7151
>             Project: Solr
>          Issue Type: Bug
>          Components: SolrJ
>            Reporter: Alan Woodward
>            Assignee: Alan Woodward
>            Priority: Minor
>             Fix For: Trunk, 5.1
>
>         Attachments: SOLR-7151.patch
>
>
> All the methods on SolrClient are declared as throwing SolrServerException (thrown if
there's an error somewhere on the server), and IOException (thrown if there's a communication
error), except for the QueryRequest methods.  These swallow up IOException and repackage them
in a SolrServerException.
> I think these are useful distinctions to make (you might want to retry on an IOException,
but not on a SolrServerException), and we should make the query methods fall in line with
the others.
> I'm not sure if this should go into 5.x as well as trunk, as it's a backwards-breaking
change.  I'm leaning towards yes, as it's a sufficiently useful API change that it's worth
the break, but I'm not going to insist on it.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org


Mime
View raw message