[ 
https://issues.apache.org/jira/browse/BLUR-54?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13594306#comment-13594306
 ] 

Aaron McCurry commented on BLUR-54:
-----------------------------------

Gagan you are correct.  The only thing I would change in your writeup of the 
problem is that findServer on TableLayout should throw a BlurException when an 
invalid shard index is passed in instead of the RuntimeException.

Aaron
                
> Improve the error when a bad shard index is provided.
> -----------------------------------------------------
>
>                 Key: BLUR-54
>                 URL: https://issues.apache.org/jira/browse/BLUR-54
>             Project: Apache Blur
>          Issue Type: Bug
>    Affects Versions: 0.2.0
>            Reporter: Tim Williams
>            Priority: Trivial
>
> Currently, if a client passes an unknown shard index, they get a message 
> about a bad connection string when in fact that's just a side effect.  I'm 
> thinking that the TableLayout implemented in ThriftBlurServer should throw a 
> RuntimeException if it isn't found - because that's where we have some useful 
> context.  Sound reasonable?  Another option is to throw it in BlurServer's 
> getConnection.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to