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

Benedict Elliott Smith edited comment on CASSANDRA-17557 at 4/21/22 9:02 AM:
-----------------------------------------------------------------------------

Thanks Ekaterina. I think there are two mistakes here in the original patch: 
the inclusion of an unnecessary property, and the failure to utilise a 
necessary property. The repair rpc timeout was intended to be kept, and 
utilised for both standard repair verbs and paxos repair verbs since if either 
fails due to transient slowdown it creates a lot of additional pressure on the 
system.

I have pushed my suggested variation on this patch 
[here|https://github.com/belliottsmith/cassandra/commits/17557-trunk]


was (Author: benedict):
Thanks Ekaterina. I think there are two mistakes here in the original patch: 
the inclusion of an unnecessary property, and the failure to utilise a 
necessary property. The repair rpc timeout was intended to be kept, and 
utilised for both standard repair verbs and paxos repair verbs.

I have pushed my suggested variation on this patch 
[here|https://github.com/belliottsmith/cassandra/commits/17557-trunk]

> Fix a few config parameters after the Paxos improvements commit
> ---------------------------------------------------------------
>
>                 Key: CASSANDRA-17557
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-17557
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Local/Config
>            Reporter: Ekaterina Dimitrova
>            Assignee: Ekaterina Dimitrova
>            Priority: Normal
>             Fix For: 4.x
>
>
> After committing the Paxos improvements, it was identified that the following 
> configuration parameters need additional work:
>  * repair_request_timeout_in_ms - can be removed
>  * paxos_auto_repair_threshold_mb - I think it can be also removed; to be 
> confirmed with the author
> Discussed a bit in Slack and on this PR - 
> https://github.com/apache/cassandra/commit/d2923275e360a1ee9db498e748c269f701bb3a8b



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to