Hello Matthew Jacobs, Adar Dembo, Todd Lipcon,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/6040
to review the following change.
Change subject: KUDU-1881: Deserializing scan token should check nullability of
column
......................................................................
KUDU-1881: Deserializing scan token should check nullability of column
Adds test cases for mismatched types and nullability on columns, and
changes the error return to InvalidStatus, since the mismatches are
indicative of changed state, and not necessarily an invalid argument.
Change-Id: I6b4c835c948723eb9bc47131e894971b8e1a2c44
---
M java/kudu-client/src/main/java/org/apache/kudu/client/KuduScanToken.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClient.java
M src/kudu/client/scan_token-internal.cc
M src/kudu/client/scan_token-test.cc
4 files changed, 181 insertions(+), 12 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/40/6040/1
--
To view, visit http://gerrit.cloudera.org:8080/6040
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6b4c835c948723eb9bc47131e894971b8e1a2c44
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Matthew Jacobs <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
