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 (#3).

Change subject: [misc] replace vector::resize() with constructor's parameter
......................................................................

[misc] replace vector::resize() with constructor's parameter

I noticed the following pattern that seems to be replicated
in various places:

  std::vector<T> v;
  v.resize(some_constant);

This patch replaces such code with calls to one of std::vector's
constructors that accepts the size of the vectors as a parameter.

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, 64 insertions(+), 79 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/84/20484/3
--
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: 3
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)

Reply via email to