isapego commented on code in PR #5253: URL: https://github.com/apache/ignite-3/pull/5253#discussion_r1963271839
########## modules/core/src/main/java/org/apache/ignite/internal/tostring/IgniteToStringBuilder.java: ########## @@ -2214,6 +2215,28 @@ private static void incValues(IdentityHashMap<Object, EntryReference> svdObjs, O } } + /** + * Converts Tuple to string representation. + * + * @param tuple Tuple. + * @return String. + */ + public static String tupleToString(Tuple tuple) { + // Keep the same as TupleImpl.toString(). Review Comment: If this is important, and we can not move this logic to a single function by some reason, let's at least add the test that compares string representation of these implementations. -- 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: notifications-unsubscr...@ignite.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org