You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Jonathan Ellis (JIRA)" <ji...@apache.org> on 2011/09/19 05:37:08 UTC
[jira] [Created] (CASSANDRA-3229) Remove ability to disable dynamic
snitch entirely
Remove ability to disable dynamic snitch entirely
-------------------------------------------------
Key: CASSANDRA-3229
URL: https://issues.apache.org/jira/browse/CASSANDRA-3229
Project: Cassandra
Issue Type: Improvement
Components: Core
Reporter: Jonathan Ellis
Assignee: Jonathan Ellis
Priority: Minor
Fix For: 1.0.0
We've moved dynamic snitch from "new, default to off" to "well tested, default to true," and it's time now to take the next step to "there is no reason to disable it, and keeping the option around just lets people shoot their foot off."
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (CASSANDRA-3229) Remove ability to disable dynamic
snitch entirely
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-3229?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jonathan Ellis updated CASSANDRA-3229:
--------------------------------------
Attachment: 3229.txt
patch removes dynamic_snitch boolean from example configuration (defaulting to true) and sets default badness threshold to 0.1
> Remove ability to disable dynamic snitch entirely
> -------------------------------------------------
>
> Key: CASSANDRA-3229
> URL: https://issues.apache.org/jira/browse/CASSANDRA-3229
> Project: Cassandra
> Issue Type: Improvement
> Components: Core
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Priority: Minor
> Fix For: 1.0.0
>
> Attachments: 3229.txt
>
>
> We've moved dynamic snitch from "new, default to off" to "well tested, default to true," and it's time now to take the next step to "there is no reason to disable it, and keeping the option around just lets people shoot their foot off."
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CASSANDRA-3229) Remove ability to disable
dynamic snitch entirely
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-3229?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13107610#comment-13107610 ]
Jonathan Ellis commented on CASSANDRA-3229:
-------------------------------------------
Ben Coverston points out that a more conservative approach would be to leave the option in but remove it from the example config file first.
> Remove ability to disable dynamic snitch entirely
> -------------------------------------------------
>
> Key: CASSANDRA-3229
> URL: https://issues.apache.org/jira/browse/CASSANDRA-3229
> Project: Cassandra
> Issue Type: Improvement
> Components: Core
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Priority: Minor
> Fix For: 1.0.0
>
>
> We've moved dynamic snitch from "new, default to off" to "well tested, default to true," and it's time now to take the next step to "there is no reason to disable it, and keeping the option around just lets people shoot their foot off."
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (CASSANDRA-3229) Remove ability to disable dynamic
snitch entirely
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-3229?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jonathan Ellis updated CASSANDRA-3229:
--------------------------------------
Reviewer: brandon.williams (was: slebresne)
> Remove ability to disable dynamic snitch entirely
> -------------------------------------------------
>
> Key: CASSANDRA-3229
> URL: https://issues.apache.org/jira/browse/CASSANDRA-3229
> Project: Cassandra
> Issue Type: Improvement
> Components: Core
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Priority: Minor
> Fix For: 1.0.0
>
> Attachments: 3229.txt
>
>
> We've moved dynamic snitch from "new, default to off" to "well tested, default to true," and it's time now to take the next step to "there is no reason to disable it, and keeping the option around just lets people shoot their foot off."
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CASSANDRA-3229) Remove ability to disable
dynamic snitch entirely
Posted by "Sylvain Lebresne (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-3229?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13107725#comment-13107725 ]
Sylvain Lebresne commented on CASSANDRA-3229:
---------------------------------------------
I'm for the more conservative approach. In general I'm not in favor of fully disabling options by fear of people shooting themselves in the foot because I think that this always end up resulting in pissing off power users at one point or the other. In the case of the DS, being able to disable it if only for debugging purposes may prove useful. But I'm totally for having hidden options that are only listed in advanced docs with the proper warnings.
> Remove ability to disable dynamic snitch entirely
> -------------------------------------------------
>
> Key: CASSANDRA-3229
> URL: https://issues.apache.org/jira/browse/CASSANDRA-3229
> Project: Cassandra
> Issue Type: Improvement
> Components: Core
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Priority: Minor
> Fix For: 1.0.0
>
>
> We've moved dynamic snitch from "new, default to off" to "well tested, default to true," and it's time now to take the next step to "there is no reason to disable it, and keeping the option around just lets people shoot their foot off."
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CASSANDRA-3229) Remove ability to disable
dynamic snitch entirely
Posted by "Brandon Williams (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-3229?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13107876#comment-13107876 ]
Brandon Williams commented on CASSANDRA-3229:
---------------------------------------------
With it defaulting to on in the last two major releases, I've never heard of anyone disabling it, so I'm fine with removing the option.
> Remove ability to disable dynamic snitch entirely
> -------------------------------------------------
>
> Key: CASSANDRA-3229
> URL: https://issues.apache.org/jira/browse/CASSANDRA-3229
> Project: Cassandra
> Issue Type: Improvement
> Components: Core
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Priority: Minor
> Fix For: 1.0.0
>
>
> We've moved dynamic snitch from "new, default to off" to "well tested, default to true," and it's time now to take the next step to "there is no reason to disable it, and keeping the option around just lets people shoot their foot off."
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CASSANDRA-3229) Remove ability to disable
dynamic snitch entirely
Posted by "Brandon Williams (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-3229?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13230465#comment-13230465 ]
Brandon Williams commented on CASSANDRA-3229:
---------------------------------------------
You can use the badness threshold from CASSANDRA-1519
> Remove ability to disable dynamic snitch entirely
> -------------------------------------------------
>
> Key: CASSANDRA-3229
> URL: https://issues.apache.org/jira/browse/CASSANDRA-3229
> Project: Cassandra
> Issue Type: Improvement
> Components: Core
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Priority: Minor
> Fix For: 1.0.0
>
> Attachments: 3229.txt
>
>
> We've moved dynamic snitch from "new, default to off" to "well tested, default to true," and it's time now to take the next step to "there is no reason to disable it, and keeping the option around just lets people shoot their foot off."
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CASSANDRA-3229) Remove ability to disable
dynamic snitch entirely
Posted by "Jonathan Ellis (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-3229?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13202404#comment-13202404 ]
Jonathan Ellis commented on CASSANDRA-3229:
-------------------------------------------
You can't have that anyway because the failure detector will force reads to a different replica when it thinks your preferred one is down.
> Remove ability to disable dynamic snitch entirely
> -------------------------------------------------
>
> Key: CASSANDRA-3229
> URL: https://issues.apache.org/jira/browse/CASSANDRA-3229
> Project: Cassandra
> Issue Type: Improvement
> Components: Core
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Priority: Minor
> Fix For: 1.0.0
>
> Attachments: 3229.txt
>
>
> We've moved dynamic snitch from "new, default to off" to "well tested, default to true," and it's time now to take the next step to "there is no reason to disable it, and keeping the option around just lets people shoot their foot off."
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CASSANDRA-3229) Remove ability to disable
dynamic snitch entirely
Posted by "Edward Capriolo (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-3229?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13201349#comment-13201349 ]
Edward Capriolo commented on CASSANDRA-3229:
--------------------------------------------
I agree that the dynamic snitch is super stable and 99.99999 % of the time everyone wants it. However it should always remain an option. I can imagine a situation where you do not want the dynamic snitch. For example a user may want to a snitch that always sends reads to predictable places. In that case not being able to disable dynamic would be bad.
> Remove ability to disable dynamic snitch entirely
> -------------------------------------------------
>
> Key: CASSANDRA-3229
> URL: https://issues.apache.org/jira/browse/CASSANDRA-3229
> Project: Cassandra
> Issue Type: Improvement
> Components: Core
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Priority: Minor
> Fix For: 1.0.0
>
> Attachments: 3229.txt
>
>
> We've moved dynamic snitch from "new, default to off" to "well tested, default to true," and it's time now to take the next step to "there is no reason to disable it, and keeping the option around just lets people shoot their foot off."
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CASSANDRA-3229) Remove ability to disable
dynamic snitch entirely
Posted by "david vanderfeesten (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-3229?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13230459#comment-13230459 ]
david vanderfeesten commented on CASSANDRA-3229:
------------------------------------------------
we are experimenting with workload split between OLTP and OLAP workload using network topology strategy. The OLTP keyspace has replicas in both (virtual)DCs. OLTP would only be written to nodes in DC1 and only being read as OLAP input in DC2. While we don't know yet if this is the right approach, with dynamic snitch off (and read repair chance:0) in case no nodes are down we hope to avoid OLAP reads to show up in DC1. Without being able to switch off dynamic snitch - I am not sure how we are able to achieve workload split (to be proven if that gives a performance benefit).
> Remove ability to disable dynamic snitch entirely
> -------------------------------------------------
>
> Key: CASSANDRA-3229
> URL: https://issues.apache.org/jira/browse/CASSANDRA-3229
> Project: Cassandra
> Issue Type: Improvement
> Components: Core
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Priority: Minor
> Fix For: 1.0.0
>
> Attachments: 3229.txt
>
>
> We've moved dynamic snitch from "new, default to off" to "well tested, default to true," and it's time now to take the next step to "there is no reason to disable it, and keeping the option around just lets people shoot their foot off."
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CASSANDRA-3229) Remove ability to disable
dynamic snitch entirely
Posted by "Brandon Williams (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-3229?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13107965#comment-13107965 ]
Brandon Williams commented on CASSANDRA-3229:
---------------------------------------------
+1
> Remove ability to disable dynamic snitch entirely
> -------------------------------------------------
>
> Key: CASSANDRA-3229
> URL: https://issues.apache.org/jira/browse/CASSANDRA-3229
> Project: Cassandra
> Issue Type: Improvement
> Components: Core
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Priority: Minor
> Fix For: 1.0.0
>
> Attachments: 3229.txt
>
>
> We've moved dynamic snitch from "new, default to off" to "well tested, default to true," and it's time now to take the next step to "there is no reason to disable it, and keeping the option around just lets people shoot their foot off."
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira