Yingchun Lai has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20291 )
Change subject: [java client] Make row errors more readable ...................................................................... [java client] Make row errors more readable Flushing data in AUTO_FLUSH_BACKGROUND mode sometimes may fail. It's possible get the error details by calling KuduSession.getPendingErrors(), but it doesn't present the information in a human-readable form. Here is an example: " Row error for primary key=[-128, 0, 0, 12], tablet=null, server=ff9a26c5fcde45f5b74c3da11a7fc89d, status=Already present: key already present (error 0) " This patch makes the row errors more readable for debugging. Here is an example after using this patch: " Row error for row=(int32 key=12, int32 column1_i=2, int32 column2_i=3, string column3_s="a string", bool column4_b=true), tablet=null, server=ba360521df2844b8a9035c7c505d30d8, status=Already present: key already present (error 0) " Change-Id: I1170986ef9b40d83a6b6da8571d15d1c6bf4df97 Reviewed-on: http://gerrit.cloudera.org:8080/20163 Reviewed-by: Yifan Zhang <[email protected]> Tested-by: Yifan Zhang <[email protected]> (cherry picked from commit 7c333560b4287c8ce399bd33768a346b39d4fd22) Reviewed-on: http://gerrit.cloudera.org:8080/20291 Reviewed-by: Yingchun Lai <[email protected]> Reviewed-by: Wang Xixu <[email protected]> Tested-by: Yingchun Lai <[email protected]> --- M java/kudu-client/src/main/java/org/apache/kudu/client/RowError.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestRowErrors.java 2 files changed, 85 insertions(+), 1 deletion(-) Approvals: Yingchun Lai: Looks good to me, approved; Verified Wang Xixu: Looks good to me, but someone else must approve -- To view, visit http://gerrit.cloudera.org:8080/20291 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: merged Gerrit-Change-Id: I1170986ef9b40d83a6b6da8571d15d1c6bf4df97 Gerrit-Change-Number: 20291 Gerrit-PatchSet: 2 Gerrit-Owner: Yingchun Lai <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Wang Xixu <[email protected]> Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]>
