Yingchun Lai has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19047 )
Change subject: KUDU-3326 [master] add improvement on interpreting the 'reserve_seconds' field for DeleteRPC ...................................................................... KUDU-3326 [master] add improvement on interpreting the 'reserve_seconds' field for DeleteRPC If the field 'reserve_seconds' is specified by the client, then it's clear the delete request coming from a newer Kudu client with the precise value for 'reserve_seconds', and the field's value from the request should be taken as-is regardless of the current setting of the --deleted_table_reserve_seconds flag at the server side. Change-Id: I604caf5d7326e27acd6d626b482376fc6eed1b0f Reviewed-on: http://gerrit.cloudera.org:8080/19047 Tested-by: Kudu Jenkins Reviewed-by: Yingchun Lai <[email protected]> --- M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java M java/kudu-client/src/main/java/org/apache/kudu/client/DeleteTableRequest.java M java/kudu-client/src/main/java/org/apache/kudu/client/KuduClient.java M src/kudu/client/client-internal.cc M src/kudu/client/client-internal.h M src/kudu/client/client-test.cc M src/kudu/client/client.cc M src/kudu/client/client.h M src/kudu/master/catalog_manager.cc M src/kudu/master/master.proto M src/kudu/tools/kudu-admin-test.cc M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/tool_action_table.cc 13 files changed, 98 insertions(+), 39 deletions(-) Approvals: Kudu Jenkins: Verified Yingchun Lai: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/19047 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I604caf5d7326e27acd6d626b482376fc6eed1b0f Gerrit-Change-Number: 19047 Gerrit-PatchSet: 15 Gerrit-Owner: KeDeng <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: KeDeng <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Reviewer: Yuqi Du <[email protected]>
