Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17518 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/17518 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- 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(-) Approvals: Alexey Serbin: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: Ia24582de6b060e908f5ecbc46e2638b95cd567b3 Gerrit-Change-Number: 17518 Gerrit-PatchSet: 5 Gerrit-Owner: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
