mrktsm commented on code in PR #4316:
URL: https://github.com/apache/cassandra/pull/4316#discussion_r2278201044
##########
test/unit/org/apache/cassandra/db/virtual/SettingsTableTest.java:
##########
@@ -71,7 +71,7 @@ public void config()
public void testArray() throws Throwable
{
Review Comment:
@Maxwell-Guo Of course! That's a great suggestion! I've added the Map test
case you requested.
The new `testMapSerialization()` test uses the exact scenario you mentioned
- `seed_provider.parameters` with a Map containing seeds and timeout values.
The test verifies that Maps get properly serialized to JSON format.
All tests are passing and the Map serialization is working as expected.
--
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
