Hello Kudu Jenkins,

I'd like you to reexamine a change.  Please visit

    http://gerrit.cloudera.org:8080/8088

to look at the new patch set (#5).

Change subject: [iwyu] update std::move
......................................................................

[iwyu] update std::move

Updated IWYU mappings to use <utility> for std::move() and std::swap()
instead of <algorithm>.  Unsorted micro-cleanups in some tests.

Also, updated the cpplint.py script from the google-styleguide
to reflect std::swap() migration from <algorithm> to <utility>. That
is to keep things consistent between 'iwyu' and 'lint/ilint' targets.
The latest version of the google-styleguide at [1] already contains the
corresponding update along with other C++11-related changes, but I would
like to bump the google-styleguide version in a separate changelist.

References:
  [1] https://github.com/google/styleguide

Change-Id: Ic5972f9a43f4c1db4a0dff5f983f7dba90c8db31
---
M build-support/iwyu/iwyu-filter.awk
M build-support/iwyu/mappings/boost-extra.imp
M build-support/iwyu/mappings/libstdcpp.imp
M src/kudu/benchmarks/tpch/tpch_real_world.cc
M src/kudu/cfile/block_cache.h
M src/kudu/cfile/bloomfile-test.cc
M src/kudu/cfile/bloomfile.cc
M src/kudu/cfile/cfile-test.cc
M src/kudu/cfile/cfile_reader.cc
M src/kudu/cfile/cfile_writer.cc
M src/kudu/client/batcher.cc
M src/kudu/client/client.cc
M src/kudu/client/meta_cache.h
M src/kudu/client/partitioner-internal.h
M src/kudu/client/replica-internal.cc
M src/kudu/client/scan_configuration.cc
M src/kudu/client/scan_token-test.cc
M src/kudu/client/schema.cc
M src/kudu/client/session-internal.cc
M src/kudu/client/table-internal.cc
M src/kudu/client/table_alterer-internal.cc
M src/kudu/client/tablet_server-internal.cc
M src/kudu/codegen/jit_wrapper.cc
M src/kudu/codegen/module_builder.cc
M src/kudu/common/partial_row.cc
M src/kudu/common/partition_pruner-test.cc
M src/kudu/common/schema.h
M src/kudu/consensus/consensus_meta-test.cc
M src/kudu/consensus/consensus_meta.cc
M src/kudu/consensus/consensus_meta_manager-stress-test.cc
M src/kudu/consensus/consensus_meta_manager.cc
M src/kudu/consensus/consensus_peers-test.cc
M src/kudu/consensus/leader_election-test.cc
M src/kudu/consensus/log.cc
M src/kudu/consensus/log_util.cc
M src/kudu/consensus/raft_consensus.h
M src/kudu/consensus/raft_consensus_quorum-test.cc
M src/kudu/fs/block_manager-stress-test.cc
M src/kudu/fs/block_manager-test.cc
M src/kudu/fs/block_manager.h
M src/kudu/fs/block_manager_util-test.cc
M src/kudu/fs/block_manager_util.cc
M src/kudu/fs/data_dirs.h
M src/kudu/fs/file_block_manager.cc
M src/kudu/fs/fs-test-util.h
M src/kudu/fs/fs_manager.cc
M src/kudu/gutil/atomicops-internals-x86.cc
M src/kudu/gutil/cpu.cc
M src/kudu/gutil/hash/city.cc
M src/kudu/gutil/ref_counted_memory.cc
M src/kudu/gutil/strings/split.cc
M src/kudu/gutil/strings/split.h
M src/kudu/gutil/strings/strcat.h
M src/kudu/gutil/strings/stringpiece.cc
M src/kudu/gutil/strings/strip.cc
M src/kudu/gutil/strings/util.cc
M src/kudu/integration-tests/exactly_once_writes-itest.cc
M src/kudu/integration-tests/external_mini_cluster-itest-base.cc
M src/kudu/integration-tests/external_mini_cluster-test.cc
M src/kudu/integration-tests/internal_mini_cluster.cc
M src/kudu/integration-tests/master-stress-test.cc
M src/kudu/integration-tests/open-readonly-fs-itest.cc
M src/kudu/integration-tests/token_signer-itest.cc
M src/kudu/integration-tests/version_migration-test.cc
M src/kudu/master/mini_master.cc
M src/kudu/master/sys_catalog-test.cc
M src/kudu/master/ts_descriptor.cc
M src/kudu/rpc/client_negotiation.h
M src/kudu/rpc/connection.h
M src/kudu/rpc/connection_id.cc
M src/kudu/rpc/exactly_once_rpc-test.cc
M src/kudu/rpc/inbound_call.h
M src/kudu/rpc/messenger.cc
M src/kudu/rpc/mt-rpc-test.cc
M src/kudu/rpc/negotiation-test.cc
M src/kudu/rpc/negotiation.cc
M src/kudu/rpc/outbound_call.cc
M src/kudu/rpc/protoc-gen-krpc.cc
M src/kudu/rpc/proxy.cc
M src/kudu/rpc/reactor.h
M src/kudu/rpc/remote_method.cc
M src/kudu/rpc/remote_user.h
M src/kudu/rpc/rpc-bench.cc
M src/kudu/rpc/rpc.h
M src/kudu/rpc/rpc_context.cc
M src/kudu/rpc/rpc_controller.cc
M src/kudu/rpc/rpc_sidecar.cc
M src/kudu/rpc/rpc_sidecar.h
M src/kudu/rpc/server_negotiation.cc
M src/kudu/rpc/server_negotiation.h
M src/kudu/rpc/service_pool.cc
M src/kudu/rpc/user_credentials.cc
M src/kudu/security/simple_acl.cc
M src/kudu/security/tls_handshake.h
M src/kudu/security/tls_socket.cc
M src/kudu/server/rpc_server.cc
M src/kudu/server/server_base.cc
M src/kudu/server/webserver.h
M src/kudu/tablet/cfile_set.h
M src/kudu/tablet/compaction-test.cc
M src/kudu/tablet/compaction.cc
M src/kudu/tablet/compaction.h
M src/kudu/tablet/delta_applier.cc
M src/kudu/tablet/delta_tracker.cc
M src/kudu/tablet/deltafile.cc
M src/kudu/tablet/multi_column_writer.cc
M src/kudu/tablet/row_op.cc
M src/kudu/tablet/rowset.cc
M src/kudu/tablet/tablet.cc
M src/kudu/tablet/tablet_bootstrap.cc
M src/kudu/tablet/tablet_history_gc-test.cc
M src/kudu/tablet/tablet_metadata.h
M src/kudu/tablet/tablet_mm_ops.cc
M src/kudu/tablet/tablet_replica-test.cc
M src/kudu/tablet/transactions/alter_schema_transaction.cc
M src/kudu/tablet/transactions/transaction_driver.cc
M src/kudu/tablet/transactions/transaction_tracker-test.cc
M src/kudu/tools/ksck_remote.h
M src/kudu/tools/tool_action-test.cc
M src/kudu/tools/tool_action_local_replica.cc
M src/kudu/tools/tool_action_pbc.cc
M src/kudu/tools/tool_action_table.cc
M src/kudu/tools/tool_action_tablet.cc
M src/kudu/tools/tool_action_wal.cc
M src/kudu/tserver/tablet_copy_client.cc
M src/kudu/tserver/tablet_copy_service-test.cc
M src/kudu/tserver/tablet_copy_source_session-test.cc
M src/kudu/tserver/tablet_copy_source_session.h
M src/kudu/tserver/tablet_server.cc
M src/kudu/twitter-demo/ingest_firehose.cc
M src/kudu/twitter-demo/insert_consumer.cc
M src/kudu/util/async_logger.h
M src/kudu/util/callback_bind-test.cc
M src/kudu/util/env-test.cc
M src/kudu/util/env.h
M src/kudu/util/env_util-test.cc
M src/kudu/util/file_cache-stress-test.cc
M src/kudu/util/jsonreader.cc
M src/kudu/util/logging.cc
M src/kudu/util/maintenance_manager.h
M src/kudu/util/mem_tracker.h
M src/kudu/util/metrics.cc
M src/kudu/util/net/net_util.cc
M src/kudu/util/pb_util-test.cc
M src/kudu/util/pb_util.cc
M src/kudu/util/process_memory.cc
M src/kudu/util/rolling_log.cc
M src/kudu/util/status-test.cc
M src/kudu/util/subprocess.cc
M src/kudu/util/test_graph.h
M src/kudu/util/thread.h
M src/kudu/util/threadpool-test.cc
M src/kudu/util/threadpool.cc
M thirdparty/download-thirdparty.sh
A thirdparty/patches/google-styleguide-cpplint.patch
155 files changed, 325 insertions(+), 347 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/88/8088/5
-- 
To view, visit http://gerrit.cloudera.org:8080/8088
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic5972f9a43f4c1db4a0dff5f983f7dba90c8db31
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot

Reply via email to