Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17535
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]> (cherry picked from commit cae3ec8df5037f560defe5b121f197f27c7065f1) --- 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/35/17535/1 -- To view, visit http://gerrit.cloudera.org:8080/17535 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.15.x Gerrit-MessageType: newchange Gerrit-Change-Id: Ia24582de6b060e908f5ecbc46e2638b95cd567b3 Gerrit-Change-Number: 17535 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
