Abhishek Chennaka has posted comments on this change. ( http://gerrit.cloudera.org:8080/20083 )
Change subject: [client]KUDU-1945 Add upsert support ...................................................................... Patch Set 1: (2 comments) I guess it makes sense to remove the test ClientTestAutoIncrementingColumn.InsertOperationNegatives altogether as we support upserts now. Removed the todo comments in python client. http://gerrit.cloudera.org:8080/#/c/20083/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/20083/1//COMMIT_MSG@7 PS1, Line 7: [client]KUDU-1945 > nit: space Done http://gerrit.cloudera.org:8080/#/c/20083/1/src/kudu/integration-tests/auto_incrementing-itest.cc File src/kudu/integration-tests/auto_incrementing-itest.cc: http://gerrit.cloudera.org:8080/#/c/20083/1/src/kudu/integration-tests/auto_incrementing-itest.cc@135 PS1, Line 135: "auto_incrementing_id" > nit: Schema::GetAutoIncrementingColumnName() Done -- To view, visit http://gerrit.cloudera.org:8080/20083 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I27a95e3a6b1d1b584cad849978313b3c8222cd3d Gerrit-Change-Number: 20083 Gerrit-PatchSet: 1 Gerrit-Owner: Abhishek Chennaka <[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-Comment-Date: Fri, 16 Jun 2023 17:57:49 +0000 Gerrit-HasComments: Yes
