Skip to content

Commit 33ca6e2

Browse files
committed
[yugabyte#6696] Leftover review comments for minor master perf tweaks
Summary: Follow up comments from D12240. Test Plan: Jenkins Reviewers: sergei Reviewed By: sergei Subscribers: ybase Differential Revision: https://phabricator.dev.yugabyte.com/D12269
1 parent cfb5885 commit 33ca6e2

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

src/yb/common/ql_rowblock.cc

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,7 @@ QLRow& QLRowBlock::Extend() {
111111
rows_.emplace_back(schema_);
112112
return rows_.back();
113113
}
114+
114115
void QLRowBlock::Reserve(size_t size) {
115116
rows_.reserve(size);
116117
}

src/yb/master/master_util.cc

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -99,9 +99,9 @@ void TakeRegistration(consensus::RaftPeerPB* source, TSInfoPB* dest) {
9999
}
100100

101101
void CopyRegistration(const consensus::RaftPeerPB& source, TSInfoPB* dest) {
102-
dest->mutable_private_rpc_addresses()->CopyFrom(source.last_known_private_addr());
103-
dest->mutable_broadcast_addresses()->CopyFrom(source.last_known_broadcast_addr());
104-
dest->mutable_cloud_info()->CopyFrom(source.cloud_info());
102+
*dest->mutable_private_rpc_addresses() = source.last_known_private_addr();
103+
*dest->mutable_broadcast_addresses() = source.last_known_broadcast_addr();
104+
*dest->mutable_cloud_info() = source.cloud_info();
105105
}
106106

107107
void TakeRegistration(ServerRegistrationPB* source, TSInfoPB* dest) {

0 commit comments

Comments
 (0)