[
https://issues.apache.org/jira/browse/CASSANDRA-17166?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17514952#comment-17514952
]
Caleb Rackliffe commented on CASSANDRA-17166:
---------------------------------------------
The patch LGTM.
There are a [few dtests that
fail|https://app.circleci.com/pipelines/github/dcapwell/cassandra/1295/workflows/0e5aba57-c9a6-4b23-81a2-a6472f7669ab/jobs/10417/tests#failed-test-4]
because of the minor change to the warning message in {{PropertiesChecker}},
but it should be pretty easy to fix that on the dtest side.
> Enhance SnakeYAML properties to be reusable outside of YAML parsing, support
> camel case conversion to snake case, and add support to ignore properties
> ------------------------------------------------------------------------------------------------------------------------------------------------------
>
> Key: CASSANDRA-17166
> URL: https://issues.apache.org/jira/browse/CASSANDRA-17166
> Project: Cassandra
> Issue Type: Improvement
> Components: Local/Config
> Reporter: David Capwell
> Assignee: David Capwell
> Priority: Normal
> Fix For: 4.x
>
> Time Spent: 12h 20m
> Remaining Estimate: 0h
>
> SnakeYaml is rather limited in the “object mapping” layer, which forces our
> internal code to match specific patterns (all fields public and camel case);
> we can remove this restriction by leveraging Jackson for property lookup, and
> leaving the YAML handling to SnakeYAML
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org
