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

Reply via email to