[
https://issues.apache.org/jira/browse/CASSANDRA-18049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17703268#comment-17703268
]
Ekaterina Dimitrova edited comment on CASSANDRA-18049 at 3/21/23 4:09 PM:
--------------------------------------------------------------------------
{quote}Only comments have been added to the patch since the last runs (which
includes circleci jdk17) 🤷🏻♀️
{quote}
Correct, there were also not many changes in trunk in those 5 days since it was
run. You are right.
J8+J11 run is all green; J11+J17 match the current snapshot of failures with
J17, published in the CASSANDRA-16895 ticket description. The respective
tickets for those failures can be also found there.
+1 to squash the comments and commit the patch to trunk. Thank you
And about:
{quote} - introduced (but not used yet), the default sync to disk mode for
chronicle-bytes is async, ref
[MappedBytesStore.java|https://github.com/OpenHFT/Chronicle-Bytes/blob/chronicle-bytes-2.23.33/src/main/java/net/openhft/chronicle/bytes/MappedBytesStore.java#L49]
-- if the user configures audit or fql logging to be blocking should we then
set {{SyncMode.SYNC}} ?
-- alternatively, if the user wants blocking AND paranoid disk sync, they can
manually set `{{{}-DmappedFile.defaultSyncMode=SYNC{}}}` (and we should
document this in cassandra.yaml){quote}
I suggest we open a follow up ticket for exploring to make the default sync
configurable from the yaml or just document it with some recommendation, etc.
(If I understood you correctly)
was (Author: e.dimitrova):
{quote}Only comments have been added to the patch since the last runs (which
includes circleci jdk17) 🤷🏻♀️
{quote}
Correct, there were also not many changes in trunk in those 5 days since it was
run. You are right.
J8+J11 run is all green; J11+J17 match the current snapshot of failures with
J17, published in the CASSANDRA-16895 ticket description. The respective
tickets for those failures can be also found there.
+1 to squash the comments and commit the patch to trunk.
And about:
{quote} - introduced (but not used yet), the default sync to disk mode for
chronicle-bytes is async, ref
[MappedBytesStore.java|https://github.com/OpenHFT/Chronicle-Bytes/blob/chronicle-bytes-2.23.33/src/main/java/net/openhft/chronicle/bytes/MappedBytesStore.java#L49]
-- if the user configures audit or fql logging to be blocking should we then
set {{SyncMode.SYNC}} ?
-- alternatively, if the user wants blocking AND paranoid disk sync, they can
manually set `{{{}-DmappedFile.defaultSyncMode=SYNC{}}}` (and we should
document this in cassandra.yaml){quote}
I suggest we open a follow up ticket for exploring to make the default sync
configurable from the yaml or just document it with some recommendation, etc.
(If I understood you correctly)
> Update Chronicle Queue
> ----------------------
>
> Key: CASSANDRA-18049
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18049
> Project: Cassandra
> Issue Type: Task
> Reporter: Ekaterina Dimitrova
> Assignee: Michael Semb Wever
> Priority: Normal
> Fix For: 5.x
>
>
> According to [this|https://chronicle.software/chronicle-support-java-17]
> article:
> {code:java}
> Starting from chronicle-bom-2.22ea26, all new releases can be run under Java
> 17 when run on the class path (but not yet under the module path).{code}
> This BOM is newer than what we currently have in both 4.1 and trunk. 4.1
> points in comments to
> [https://mvnrepository.com/artifact/net.openhft/chronicle-bom/1.16.23] which
> I believe was just forgotten to be updated/removed. The versions we see
> correspond to this BOM
> [https://mvnrepository.com/artifact/net.openhft/chronicle-bom/2.20.226]
> It is still older than chronicle-bom-2.22ea26 so we need to upgrade. I
> suggest we also add a comment again which BOM is considered, this makes
> things easier.
> Further to running CI, review of the CHANGE logs needs to happen to ensure we
> do not miss anything that can impact us and it is not caught by our tests.
> For testing with JDK17, please, contact [~e.dimitrova] for latest branch and
> CI config (at this point feature branch in the cassandra repo does not exist)
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
