Hello Alexey Serbin, Yifan Zhang, Kudu Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/20163

to look at the new patch set (#6).

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
---
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(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/63/20163/6
--
To view, visit http://gerrit.cloudera.org:8080/20163
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I1170986ef9b40d83a6b6da8571d15d1c6bf4df97
Gerrit-Change-Number: 20163
Gerrit-PatchSet: 6
Gerrit-Owner: Wang Xixu <1450306...@qq.com>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Wang Xixu <1450306...@qq.com>
Gerrit-Reviewer: Yifan Zhang <chinazhangyi...@163.com>

Reply via email to