[
https://issues.apache.org/jira/browse/CASSANDRA-17650?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17559118#comment-17559118
]
Andres de la Peña commented on CASSANDRA-17650:
-----------------------------------------------
Looks good to me, nothing else to add. I have rebased the patch and applied to
4.1 without conflicts, CI is running:
||PR||CI||
|[4.1|https://github.com/adelapena/cassandra/tree/17650-4.1-review]|[j8|https://app.circleci.com/pipelines/github/adelapena/cassandra/1730/workflows/f7d5adcc-f4f0-4c9f-ab9d-a0be4140f58d]
[j11|https://app.circleci.com/pipelines/github/adelapena/cassandra/1730/workflows/afa9f339-64e3-4a58-9fbf-b5c05b6a24f1]|
|[trunk|https://github.com/adelapena/cassandra/tree/17650-trunk-review]|[j8|https://app.circleci.com/pipelines/github/adelapena/cassandra/1731/workflows/1d855d29-a32d-4056-b5b1-3a8f3ec14ab8]
[j11|https://app.circleci.com/pipelines/github/adelapena/cassandra/1731/workflows/b57a5e69-d01d-447f-9de1-bae9110becb1]|
> Check startup and setter validations for batch_size_fail_threshold
> ------------------------------------------------------------------
>
> Key: CASSANDRA-17650
> URL: https://issues.apache.org/jira/browse/CASSANDRA-17650
> Project: Cassandra
> Issue Type: Bug
> Components: Local/Config
> Reporter: Ekaterina Dimitrova
> Assignee: Akin Tekeoglu
> Priority: Low
> Fix For: 4.1-beta, 4.1.x, 4.x
>
> Time Spent: 2h 40m
> Remaining Estimate: 0h
>
> During manual inspection it was noticed that batch_size_warn_threshold goes
> through checkValidForByteConversion, but this is not the case for
> batch_size_fail_threshold.
> This ticket is opened to check whether this is a bug or not.
> If it is confirmed it is a bug it needs to be fixed and the ticket can be
> changed to type bug. It seems that this validation was added as part of
> CASSANDRA-15470 to 4.0+. Maybe we also need it in 3.0 and 3.11?
> CC [~adelapena]
--
This message was sent by Atlassian Jira
(v8.20.7#820007)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
