[ 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