Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15005 )
Change subject: clock: remove shared ownership ...................................................................... Patch Set 3: (5 comments) http://gerrit.cloudera.org:8080/#/c/15005/3/src/kudu/consensus/consensus_queue.h File src/kudu/consensus/consensus_queue.h: http://gerrit.cloudera.org:8080/#/c/15005/3/src/kudu/consensus/consensus_queue.h@572 PS3, Line 572: scoped_refptr<TimeManager> time_manager_; Is it possible to get rid of shared ownership for this member as well? It seems TimeManager instance gets here from calling RaftConsensus::Start() in tablet_server.cc, where and there it doesn't appear to be shared among multiple instances of RaftConsensus objects. http://gerrit.cloudera.org:8080/#/c/15005/3/src/kudu/integration-tests/ts_recovery-itest.cc File src/kudu/integration-tests/ts_recovery-itest.cc: http://gerrit.cloudera.org:8080/#/c/15005/3/src/kudu/integration-tests/ts_recovery-itest.cc@593 PS3, Line 593: () nit: drop parentheses? http://gerrit.cloudera.org:8080/#/c/15005/3/src/kudu/server/server_base.h File src/kudu/server/server_base.h: http://gerrit.cloudera.org:8080/#/c/15005/3/src/kudu/server/server_base.h@104 PS3, Line 104: nit: add const specifier? http://gerrit.cloudera.org:8080/#/c/15005/3/src/kudu/server/server_base.cc File src/kudu/server/server_base.cc: http://gerrit.cloudera.org:8080/#/c/15005/3/src/kudu/server/server_base.cc@387 PS3, Line 387: () nit: drop the parentheses? http://gerrit.cloudera.org:8080/#/c/15005/3/src/kudu/tablet/mvcc-test.cc File src/kudu/tablet/mvcc-test.cc: http://gerrit.cloudera.org:8080/#/c/15005/3/src/kudu/tablet/mvcc-test.cc@185 PS3, Line 185: () nit: drop the parentheses? -- To view, visit http://gerrit.cloudera.org:8080/15005 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3e75a539c5de1c367d05784f544b96197249ec49 Gerrit-Change-Number: 15005 Gerrit-PatchSet: 3 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Fri, 10 Jan 2020 18:47:14 +0000 Gerrit-HasComments: Yes
