Hello Tidy Bot, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16776
to look at the new patch set (#5).
Change subject: [build] Support and require C++17
......................................................................
[build] Support and require C++17
This patch adjust the build to start supporting and requiring C++17
(therefore GCC 7+). It adjusts the documentation and supported
OS matrix to match the new requirements as well.
I needed to replace std::random_shuffle with std::shuffle.
In C++14 `std::random_shuffle` is deprectated and in C++17 it is
removed all together. This patch replaces `std::random_shuffle` with
`std::shuffle` passing in a generator.
Additionally I needed to patch libev due to a small C++17 incompatibility:
https://github.com/enki/libev/pull/8
Change-Id: I15312473bda8c743eac228cd23c337fae5acf99d
---
M CMakeLists.txt
M README.adoc
R build-support/ccache-devtoolset-7/c++
R build-support/ccache-devtoolset-7/cc
M build-support/enable_devtoolset.sh
M build-support/enable_devtoolset_inner.sh
M docker/Dockerfile
M docker/bootstrap-dev-env.sh
M docs/installation.adoc
M src/kudu/benchmarks/wal_hiccup.cc
M src/kudu/client/client-test.cc
M src/kudu/client/predicate-test.cc
M src/kudu/client/symbols.map
M src/kudu/common/wire_protocol-test.cc
M src/kudu/fs/log_block_manager-test.cc
M src/kudu/integration-tests/all_types-itest.cc
M src/kudu/integration-tests/ts_itest-base.cc
M src/kudu/master/catalog_manager.cc
M src/kudu/tablet/compaction-test.cc
M src/kudu/tablet/tablet.cc
M src/kudu/transactions/txn_status_manager-test.cc
M src/kudu/tserver/tablet_server-test.cc
M thirdparty/build-definitions.sh
M thirdparty/build-thirdparty.sh
M thirdparty/download-thirdparty.sh
A thirdparty/patches/libev-c17.patch
M thirdparty/preflight.py
27 files changed, 137 insertions(+), 93 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/76/16776/5
--
To view, visit http://gerrit.cloudera.org:8080/16776
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I15312473bda8c743eac228cd23c337fae5acf99d
Gerrit-Change-Number: 16776
Gerrit-PatchSet: 5
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
