[
https://issues.apache.org/jira/browse/CASSANDRA-15234?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17470082#comment-17470082
]
Ekaterina Dimitrova edited comment on CASSANDRA-15234 at 1/6/22, 5:43 PM:
--------------------------------------------------------------------------
Moved back to 4.x as the patch has backward compatibility, introducing no
breaking changes which aligns with our release discussions.
{quote}r on if we allow a single space between values and units. (ex. Allow
both 10MiB and 10 MiB
{quote}
This is very easy to introduce but I think it is confusing to have both,
similar to what we decided about the letter case. I would prefer to keep the
current version only. Let me know what you think.
{quote}The second is trying, where possible, to leave the cases where we throw
{{ConfigurationException}} intact. (This mostly applies to the cases where we
might throw it via JMX, i.e. we have something of a contract at the moment.)
{quote}
Good point, I will handle it, thanks.
was (Author: e.dimitrova):
Moved back to 4.x as the patch has backward compatibility, introducing no
breaking changes which aligns with our release discussions.
{quote}r on if we allow a single space between values and units. (ex. Allow
both {{10MiB }}and {{10 MiB}}
{quote}
This is very easy to introduce but I think it is confusing to have both,
similar to what we decided about the letter case. I would prefer to keep the
current version only. Let me know what you think.
{quote}bq. The second is trying, where possible, to leave the cases where we
throw {{ConfigurationException}} intact. (This mostly applies to the cases
where we might throw it via JMX, i.e. we have something of a contract at the
moment.)
{quote}
Good point, I will handle it, thanks.
> Standardise config and JVM parameters
> -------------------------------------
>
> Key: CASSANDRA-15234
> URL: https://issues.apache.org/jira/browse/CASSANDRA-15234
> Project: Cassandra
> Issue Type: Bug
> Components: Local/Config
> Reporter: Benedict Elliott Smith
> Assignee: Ekaterina Dimitrova
> Priority: Normal
> Fix For: 4.x
>
> Attachments: CASSANDRA-15234-3-DTests-JAVA8.txt
>
>
> We have a bunch of inconsistent names and config patterns in the codebase,
> both from the yams and JVM properties. It would be nice to standardise the
> naming (such as otc_ vs internode_) as well as the provision of values with
> units - while maintaining perpetual backwards compatibility with the old
> parameter names, of course.
> For temporal units, I would propose parsing strings with suffixes of:
> {{code}}
> u|micros(econds?)?
> ms|millis(econds?)?
> s(econds?)?
> m(inutes?)?
> h(ours?)?
> d(ays?)?
> mo(nths?)?
> {{code}}
> For rate units, I would propose parsing any of the standard {{B/s, KiB/s,
> MiB/s, GiB/s, TiB/s}}.
> Perhaps for avoiding ambiguity we could not accept bauds {{bs, Mbps}} or
> powers of 1000 such as {{KB/s}}, given these are regularly used for either
> their old or new definition e.g. {{KiB/s}}, or we could support them and
> simply log the value in bytes/s.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org
