Kousuke Saruta has posted comments on this change. Change subject: Remove Schema#toString from RowResult#toString ......................................................................
Patch Set 1: (1 comment) Thanks for the comment. At first, I tried to add the row data but is it already done by toStringLongFormat right? If it's better to add it to toString too, I'll fix it. http://gerrit.cloudera.org:8080/#/c/5534/1/java/kudu-client/src/main/java/org/apache/kudu/client/RowResult.java File java/kudu-client/src/main/java/org/apache/kudu/client/RowResult.java: Line 510: return "RowResult index: " + this.index + ", size: " + this.rowSize; > makes sense, though I wonder if we should replace this entirely with a stri Thanks for the comment. At first, I tried to add the row data but is it already done by toStringLongFormat right? If it's better to add it to toString too, I'll fix it. -- To view, visit http://gerrit.cloudera.org:8080/5534 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id2e01cc1cc52958c73dfd75304a826ea033ce785 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Kousuke Saruta <[email protected]> Gerrit-Reviewer: Kousuke Saruta <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
