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 (#3).

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/TestAsyncKuduSession.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
5 files changed, 57 insertions(+), 17 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/18/17518/3
--
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: 3
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)

Reply via email to