Hello Alexey Serbin, Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/17518 to look at the new patch set (#4). Change subject: [java] KUDU-3267 Improve logging on writes to non-existent partitions ...................................................................... [java] KUDU-3267 Improve logging on writes to non-existent partitions This patch helps to log a more meaningful error message when a non-existent range partition is written to by the Kudu Java client. Due to different behavior of the client in different flush modes, the existing messages returned by the client are different when different flush modes are used. Effort has been made to bring consistency to the error messages returned in all the flush modes as well. For example in the default AUTO_FLUSH_BACKGROUND mode Existing log message snippet: Not found: ([0x000000018005BDAA4BF52400, 0x000000028005BCC2F4C85400)) New log message snippet (in any FLUSH mode): Not Found: accessed range partition ([0x80000064, 0x800000C8)) does not exist in table: TestKuduSession Thanks to Grant Henke for the help in writing this patch. Change-Id: Ia24582de6b060e908f5ecbc46e2638b95cd567b3 --- M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java M java/kudu-client/src/main/java/org/apache/kudu/client/NonCoveredRangeException.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduPartitioner.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduSession.java 4 files changed, 59 insertions(+), 13 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/18/17518/4 -- To view, visit http://gerrit.cloudera.org:8080/17518 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ia24582de6b060e908f5ecbc46e2638b95cd567b3 Gerrit-Change-Number: 17518 Gerrit-PatchSet: 4 Gerrit-Owner: Abhishek Chennaka <achenn...@cloudera.com> Gerrit-Reviewer: Abhishek Chennaka <achenn...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120)