Hello Tidy Bot, Kudu Jenkins, Abhishek Chennaka,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/20484
to look at the new patch set (#2).
Change subject: [misc] avoid calling vector::resize() right away
......................................................................
[misc] avoid calling vector::resize() right away
I noticed the following pattern that seems to be replicated
in various places:
std::vector<T> v;
v.resize(some_constant);
To stop the replication of this funny pattern, this patch removes its
current occurrences.
I also took the liberty of updating the related code to be more
style-compliant and making other minor improvements.
This patch doesn't contain any functional modifications.
Change-Id: Id72689597c39b66a6f2f24f734a12a90afef152a
---
M src/kudu/benchmarks/wal_hiccup.cc
M src/kudu/cfile/encoding-test.cc
M src/kudu/client/master_rpc.cc
M src/kudu/client/schema.cc
M src/kudu/common/partition.cc
M src/kudu/common/rowblock.cc
M src/kudu/experiments/merge-test.cc
M src/kudu/fs/log_block_manager.cc
M src/kudu/master/txn_manager-test.cc
M src/kudu/rpc/rpc_stub-test.cc
M src/kudu/rpc/server_negotiation.cc
M src/kudu/security/tls_handshake-test.cc
M src/kudu/tablet/compaction.cc
M src/kudu/tablet/delta_compaction.cc
M src/kudu/tablet/deltafile-test.cc
M src/kudu/tablet/deltamemstore-test.cc
M src/kudu/tablet/lock_manager.cc
M src/kudu/tablet/rowset_tree.cc
M src/kudu/tablet/tablet-test-base.h
M src/kudu/tablet/tablet_random_access-test.cc
M src/kudu/tserver/tablet_copy_source_session.cc
M src/kudu/util/maintenance_manager.cc
M src/kudu/util/rle-test.cc
M src/kudu/util/threadpool-test.cc
M src/kudu/util/url-coding-test.cc
25 files changed, 40 insertions(+), 67 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/84/20484/2
--
To view, visit http://gerrit.cloudera.org:8080/20484
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id72689597c39b66a6f2f24f734a12a90afef152a
Gerrit-Change-Number: 20484
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
