[jira] [Commented] (SOLR-8062) Solr should raise an exception if minRf is specified and not achieved for an update request

2015-11-08 Thread Shai Erera (JIRA)

[ 
https://issues.apache.org/jira/browse/SOLR-8062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14995639#comment-14995639
 ] 

Shai Erera commented on SOLR-8062:
--

I wanted to share a possible confusion on the user end, if we start throwing 
exceptions.

Since Solr (and Lucene for that matter) is unable to rollback a specific update 
request, I believe that's the reason why 'minRF' is used only as a hint, and 
telling the user that the update request didn't achieve it. By not throwing the 
exception, the user/client couldn't _think_ that the update request was 
unsuccessful, and in fact, I believe that in many cases the update itself did 
reach the failed replicas eventually.

If we start throwing an exception, I worry that users might think that the 
update request was unsuccessful as a whole. If they care, they might re-issue 
the request. But if they don't, then they might not realize that the update 
*did* get through, and even reach the failed replicas eventually. This might 
cause 'surprises' and inconsistency between what the user thinks is in the 
index and what isn't.

I personally think that an exception is better than just a warning in the 
response, but wanted to raise this potential confusion, in case others have an 
opinion on it. Perhaps if we throw an explicit exception it will be easier to 
note by users, i.e. {{UpdateNotFullyAppliedException}} or something like that. 
It's not a total update failure, but it is some sort of failure cause we 
weren't able to meet the client's requirements.

> Solr should raise an exception if minRf is specified and not achieved for an 
> update request
> ---
>
> Key: SOLR-8062
> URL: https://issues.apache.org/jira/browse/SOLR-8062
> Project: Solr
>  Issue Type: Improvement
>  Components: SolrCloud
>Reporter: Timothy Potter
>
> See discussion in SOLR-8034



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

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



[jira] [Commented] (SOLR-8062) Solr should raise an exception if minRf is specified and not achieved for an update request

2015-11-08 Thread Mark Miller (JIRA)

[ 
https://issues.apache.org/jira/browse/SOLR-8062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14995655#comment-14995655
 ] 

Mark Miller commented on SOLR-8062:
---

We have to throw an exception because the request as designed could not be 
completed. Of course we wil document what that exception means. 

> Solr should raise an exception if minRf is specified and not achieved for an 
> update request
> ---
>
> Key: SOLR-8062
> URL: https://issues.apache.org/jira/browse/SOLR-8062
> Project: Solr
>  Issue Type: Improvement
>  Components: SolrCloud
>Reporter: Timothy Potter
>
> See discussion in SOLR-8034



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

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



[jira] [Commented] (SOLR-8062) Solr should raise an exception if minRf is specified and not achieved for an update request

2015-09-22 Thread Anshum Gupta (JIRA)

[ 
https://issues.apache.org/jira/browse/SOLR-8062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14902914#comment-14902914
 ] 

Anshum Gupta commented on SOLR-8062:


I agree with this. Specially, considering that now LIR doesn't kick in if the 
minRF isn't  achieved, this makes all the more sense.


> Solr should raise an exception if minRf is specified and not achieved for an 
> update request
> ---
>
> Key: SOLR-8062
> URL: https://issues.apache.org/jira/browse/SOLR-8062
> Project: Solr
>  Issue Type: Improvement
>  Components: SolrCloud
>Reporter: Timothy Potter
>
> See discussion in SOLR-8034



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

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



[jira] [Commented] (SOLR-8062) Solr should raise an exception if minRf is specified and not achieved for an update request

2015-09-22 Thread Mark Miller (JIRA)

[ 
https://issues.apache.org/jira/browse/SOLR-8062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14902569#comment-14902569
 ] 

Mark Miller commented on SOLR-8062:
---

For 6 we might even want to consider having this set by default to the number 
of replicas and let users scale back to less safety instead of the reverse.

> Solr should raise an exception if minRf is specified and not achieved for an 
> update request
> ---
>
> Key: SOLR-8062
> URL: https://issues.apache.org/jira/browse/SOLR-8062
> Project: Solr
>  Issue Type: Improvement
>  Components: SolrCloud
>Reporter: Timothy Potter
>
> See discussion in SOLR-8034



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

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



[jira] [Commented] (SOLR-8062) Solr should raise an exception if minRf is specified and not achieved for an update request

2015-09-16 Thread Mark Miller (JIRA)

[ 
https://issues.apache.org/jira/browse/SOLR-8062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14790554#comment-14790554
 ] 

Mark Miller commented on SOLR-8062:
---

My feeling is, if you asked for a min rep and it was not achieved, the request 
failed. We should not just hint to the client, but disrupt its flow. If you are 
using a min rep param, presumably, this is the behavior you would want.

> Solr should raise an exception if minRf is specified and not achieved for an 
> update request
> ---
>
> Key: SOLR-8062
> URL: https://issues.apache.org/jira/browse/SOLR-8062
> Project: Solr
>  Issue Type: Improvement
>  Components: SolrCloud
>Reporter: Timothy Potter
>
> See discussion in SOLR-8034



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

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