[
https://issues.apache.org/jira/browse/CASSANDRA-17161?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17749140#comment-17749140
]
Andres de la Peña commented on CASSANDRA-17161:
-----------------------------------------------
The fix with the {{INTEGER_PRIMITIVE_LONG}} converter makes sense to me.
As for the removal of the {{commitlog_sync_batch_window_in_ms}} property, I
don't know if we have an agreement on how to remove unused old properties. An
upgrade from 4.1 having a yaml containing that property will fail with the
current patch. This could be understood as breaking backward compatibility. I
guess the alternative approach would be keeping the old property ad aeternum
and logging a warning saying that the property is going to be ignored. I don't
know if we have already had that discussion on the dev list and I have missed
it but, if not, we should probably have it now.
> Remove commitlog_sync_batch_window_in_ms
> ----------------------------------------
>
> Key: CASSANDRA-17161
> URL: https://issues.apache.org/jira/browse/CASSANDRA-17161
> Project: Cassandra
> Issue Type: Task
> Components: Build
> Reporter: Ekaterina Dimitrova
> Assignee: Stefan Miklosovic
> Priority: Normal
> Fix For: 5.x
>
> Time Spent: 10m
> Remaining Estimate: 0h
>
> commitlog_sync_batch_window_in_ms is deprecated in 4.0 and has to be removed
> in 5.0
> This will require also some work around the in-jvm upgrade tests.
> Currently they do not work if we remove a parameter in a later version.
> Side note: They also don't work if we change the type of the parameter in
> Config class in a newer Cassandra version. If you plan to set it in the tests
> themselves, it won't work. If you change the type but your don't set the
> parameter, no issue so there might be some workaround solution instead of
> working on the dtest framework itself.... TBD
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
