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

Alex Petrov commented on CASSANDRA-15708:
-----------------------------------------

Thank you for the patch! +1 from my side. The only (minor) suggestion is to 
maybe warn/log on the missing field. If we want to bring previous behaviour of 
failing/non failing on config fields, we shouold change `Config` API. But I'm 
not sure if it's worth doing. If you like suggestion, you can just do it on 
commit (or ignore otherwise).

> Fix in-jvm upgrade dtests
> -------------------------
>
>                 Key: CASSANDRA-15708
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-15708
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Test/dtest
>            Reporter: Marcus Eriksson
>            Assignee: Marcus Eriksson
>            Priority: Normal
>
> In-jvm upgrade dtests were broken by CASSANDRA-15539



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to