Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/4985
Change subject: IMPALA-3710: Kudu DML should ignore conflicts, pt2 ...................................................................... IMPALA-3710: Kudu DML should ignore conflicts, pt2 Second part of IMPALA-3710, which removed the IGNORE DML option and changed the following errors on Kudu DML operations to be ignored: * INSERT where the PK already exists * UPDATE/DELETE where the PK doesn't exist This changes other data-related errors to be ignored as well: * NULLs in non-nullable columns, i.e. null constraint violoations. * Rows with PKs that are in an 'uncovered range'. This also adds some metadata to the thrift report sent to the coordinator from sinks so the total number of rows with errors can be added to the profile. An upcoming change will add this new info to the beeswax interface and show it in the shell output (IMPALA-3713). Testing: Updated kudu_crud tests to check the number of rows with errors. Change-Id: I4eb1ad91dc355ea51de261c3a14df0f9d28c879c --- M be/src/exec/data-sink.cc M be/src/exec/data-sink.h M be/src/exec/hdfs-table-sink.cc M be/src/exec/kudu-table-sink.cc M be/src/exec/kudu-table-sink.h M be/src/runtime/coordinator.cc M common/thrift/ImpalaInternalService.thrift M common/thrift/generate_error_codes.py M testdata/workloads/functional-query/queries/QueryTest/kudu_crud.test 9 files changed, 174 insertions(+), 98 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/85/4985/1 -- To view, visit http://gerrit.cloudera.org:8080/4985 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I4eb1ad91dc355ea51de261c3a14df0f9d28c879c Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Matthew Jacobs <m...@cloudera.com>
