maedhroz commented on code in PR #3305:
URL: https://github.com/apache/cassandra/pull/3305#discussion_r1611872166


##########
src/java/org/apache/cassandra/utils/PojoToString.java:
##########
@@ -120,7 +120,7 @@ public static String pojoMapToString(Map<String, Object> 
map, String formatStrin
         return pojoToString(map, formatString);
     }
 
-    private static String pojoToString(Object obj, String formatString)
+    public static String pojoToString(Object obj, String formatString)

Review Comment:
   I'm guessing it's because it starts as a string in 
`listConsensusMigrations()` and we just didn't convert earlier. For my patch, 
it looks like I may actually be able to avoid `PojoToString` altogether, 
looking at the comments above...



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to