Yingchun Lai has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20227 )
Change subject: [server] KUDU-1945 Auto_incrementing column UPSERT support ...................................................................... [server] KUDU-1945 Auto_incrementing column UPSERT support Add UPSERT operation support on the server side when the entire row is present in the request including the auto-incrementing column. There will be followup patches to update the clients and test the corresponding changes as well. With this patch the expected behavior for different operations from Kudu clients is as follows: Operation Presence of auto-incrementing column INSERT/INSERT_IGNORE Should not be specified UPDATE/UPDATE_IGNORE Has to be specified UPSERT/UPSERT_IGNORE Has to be specified DELETE/DELETE_IGNORE Has to be specified This implementation of UPSERT operation behavior in this patch is to facilitate incremental restores consistently when using Kudu backup and restore tool. More details in the design document: https://docs.google.com/document/d/1-x6F5TkRYMqt2umDBQ5x-Aa4dqKwFYpbxo1A6QoMafo/edit#heading=h.vnagsa4oxwrf Change-Id: Ib5cc4d80f77c165452572948f68c76fc70394d47 Reviewed-on: http://gerrit.cloudera.org:8080/20061 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Alexey Serbin <[email protected]> (cherry picked from commit cbbb60e532e388665480f1c8be383305eb8fae62) Reviewed-on: http://gerrit.cloudera.org:8080/20227 Reviewed-by: Yingchun Lai <[email protected]> Tested-by: Yingchun Lai <[email protected]> --- M src/kudu/common/row_operations.cc M src/kudu/tablet/tablet_auto_incrementing-test.cc 2 files changed, 202 insertions(+), 49 deletions(-) Approvals: Yingchun Lai: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/20227 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: merged Gerrit-Change-Id: Ib5cc4d80f77c165452572948f68c76fc70394d47 Gerrit-Change-Number: 20227 Gerrit-PatchSet: 4 Gerrit-Owner: Yingchun Lai <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]>
