Dan Burkert has submitted this change and it was merged.

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
Reviewed-on: http://gerrit.cloudera.org:8080/6040
Reviewed-by: Adar Dembo <[email protected]>
Tested-by: Kudu Jenkins
---
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, 180 insertions(+), 12 deletions(-)

Approvals:
  Adar Dembo: Looks good to me, approved
  Kudu Jenkins: Verified



-- 
To view, visit http://gerrit.cloudera.org:8080/6040
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I6b4c835c948723eb9bc47131e894971b8e1a2c44
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Matthew Jacobs <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>

Reply via email to