mrktsm opened a new pull request, #4316: URL: https://github.com/apache/cassandra/pull/4316
**JIRA:** https://issues.apache.org/jira/browse/CASSANDRA-20827 Replace Collections.toString() with JSON serialization for arrays, maps, and collections in SettingsTable.getValue() to improve programmatic access. Complex configuration values were previously displayed using Collections.toString() format: ``` seed_provider.parameters | {seeds=127.0.0.1:7000} ``` Now output as proper JSON for better programmatic parsing: ``` seed_provider.parameters | {"seeds": "127.0.0.1:7000"} ``` **Changes:** - Modified SettingsTable.getValue() to use JsonUtils.JSON_OBJECT_MAPPER for complex types - Added exception handling with fallback to toString() for robustness - Updated all affected tests to expect JSON array format: ["item1", "item2"] vs [item1, item2] - Covers data_file_directories, cipher_suites, and protocol configuration tests **Testing:** - All existing tests updated and passing with new JSON format - Verified proper JSON serialization for arrays, maps, and collections - Exception handling tested via fallback mechanism patch by Marko Tsymbaliuk for CASSANDRA-20827 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org For additional commands, e-mail: pr-h...@cassandra.apache.org
