Mahesh Reddy has posted comments on this change. ( http://gerrit.cloudera.org:8080/16703 )
Change subject: [client] use make_shared for a few call sites ...................................................................... Patch Set 1: Code-Review+1 (2 comments) http://gerrit.cloudera.org:8080/#/c/16703/1/src/kudu/client/client-internal.cc File src/kudu/client/client-internal.cc: http://gerrit.cloudera.org:8080/#/c/16703/1/src/kudu/client/client-internal.cc@563 PS1, Line 563: unique_ptr<Schema> new_schema(new Schema); > Semantically, there isn't any difference between 'Schema()' and 'Schema' in Ack http://gerrit.cloudera.org:8080/#/c/16703/1/src/kudu/client/meta_cache.h File src/kudu/client/meta_cache.h: http://gerrit.cloudera.org:8080/#/c/16703/1/src/kudu/client/meta_cache.h@19 PS1, Line 19: #pragma once > We slowly migrating existing code w.r.t. replacing the include guards with Sure I can do that. I didn't know that we slowly migrate the include guard changes but I'll post a patch soon. -- To view, visit http://gerrit.cloudera.org:8080/16703 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I0c1b8bdfffec596db6a262c195cc94796d1bc1d5 Gerrit-Change-Number: 16703 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]> Gerrit-Comment-Date: Mon, 09 Nov 2020 23:59:00 +0000 Gerrit-HasComments: Yes
