[ https://issues.apache.org/jira/browse/CASSANDRA-19583?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17842046#comment-17842046 ]
Jon Haddad commented on CASSANDRA-19583: ---------------------------------------- To be clear, I'm suggesting we make "0" work, without a unit. I'm not suggesting we change how 0MiB/s works. They can be equivalent. Making it mandatory to supply a unit with 0 is a weird user experience. Zero is zero, there's no meaning to the label, it's superfluous. > setting compaction throughput to 0 throws a startup error > --------------------------------------------------------- > > Key: CASSANDRA-19583 > URL: https://issues.apache.org/jira/browse/CASSANDRA-19583 > Project: Cassandra > Issue Type: Bug > Components: Local/Config > Reporter: Jon Haddad > Priority: Normal > Fix For: 4.1.x, 5.0.x, 5.x > > > The inline docs say: > {noformat} > Setting this to 0 disables throttling. > {noformat} > However, on startup, we throw this error: > {noformat} > Caused by: java.lang.IllegalArgumentException: Invalid data rate: 0 Accepted > units: MiB/s, KiB/s, B/s where case matters and only non-negative values a> > Apr 23 23:12:01 cassandra0 cassandra[3424]: at > org.apache.cassandra.config.DataRateSpec.<init>(DataRateSpec.java:52) > Apr 23 23:12:01 cassandra0 cassandra[3424]: at > org.apache.cassandra.config.DataRateSpec.<init>(DataRateSpec.java:61) > Apr 23 23:12:01 cassandra0 cassandra[3424]: at > org.apache.cassandra.config.DataRateSpec$LongBytesPerSecondBound.<init>(DataRateSpec.java:232) > Apr 23 23:12:01 cassandra0 cassandra[3424]: ... 27 common frames > omitted > {noformat} > We should allow 0 as per the inline doc. -- This message was sent by Atlassian Jira (v8.20.10#820010) --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org