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

Ekaterina Dimitrova edited comment on CASSANDRA-15234 at 12/2/21, 11:47 PM:
----------------------------------------------------------------------------

Short update:
 - I figured out what needs to be done in order to be able to test with our own 
CCM branch - ticket to update our docs opened CASSANDRA-17182.
 - I was also thinking of adding a test to the ccm tests but it seems they are 
not run in CI, not sure what is the story there. ([~mck], any ideas?)
 - Almost all tests pass now, it seems I have an issue with the assert warnings 
only(a few Python DTests failed), I have to correct the string pattern used. I 
did it 
[here|https://github.com/apache/cassandra-dtest/pull/169/commits/7c417847092524d46de9e03d11d6acc63159e2dd]
 but I haven't run whole CI for this now. [Java8 
CI|https://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra/1239/workflows/345d82ca-1e2a-4db9-b660-a88324793a40]
 | [Java11 CI 
|https://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra/1239/workflows/d0f8f584-ede1-42f1-96ad-9fbc2d981a56].
 This branch was not rebased in the past two weeks so some of the test fixes 
are not applied.
 - I will do the JMX change next week in a new commit, it also shouldn't bother 
any reviews to happen in the meantime if/when people are available. I know 
[~dcapwell] and [~blerer] are busy and also holidays are approaching.


was (Author: e.dimitrova):
Short update:
- I figured out what needs to be done in order to be able to test with our own 
CCM branch - ticket to update our docs opened CASSANDRA-17182.
- I was also thinking of adding a test to the ccm tests but it seems they are 
not run in CI, not sure what is the story there. ([~mck], any ideas?)
- Almost all tests pass now, it seems I have an issue with the assert warnings 
only, I have to correct the pattern. I did it 
[here|https://github.com/apache/cassandra-dtest/pull/169/commits/7c417847092524d46de9e03d11d6acc63159e2dd]
 but I don't think I haven't run whole CI for this. [Java8 
CI|https://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra/1239/workflows/345d82ca-1e2a-4db9-b660-a88324793a40]
 | [Java11 
CIhttps://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra/1239/workflows/d0f8f584-ede1-42f1-96ad-9fbc2d981a56].
 This branch was not rebased in the past two weeks so some of the test fixes 
are no applied. 
- I will do the JMX change next week in a new commit, it also shouldn't bother 
any reviews to happen in the meantime if/when people are available. I know 
[~dcapwell] and [~blerer] are busy and also holidays are approaching. 

> 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: 5.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

Reply via email to