Hello Alexey Serbin, Adar Dembo,

I'd like you to do a code review. Please visit

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

to review the following change.


Change subject: build: don't use constexpr lambdas
......................................................................

build: don't use constexpr lambdas

We use constexpr lambdas, and some compilers are fine with it.
Officially, they aren't supported until C++17, so this patch replaces
such instances with const lambdas.

This would yield a build failure in some environments.

See https://en.cppreference.com/w/cpp/language/lambda for more details
about constexpr lambdas in C++17.

Change-Id: I9d1bdb84d9e6ee5c6a4a920d46f5daee68975208
---
M src/kudu/tablet/compaction_policy.cc
M src/kudu/tools/rebalance_algo-test.cc
M src/kudu/tserver/tablet_server-test.cc
3 files changed, 3 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/70/11970/1
--
To view, visit http://gerrit.cloudera.org:8080/11970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9d1bdb84d9e6ee5c6a4a920d46f5daee68975208
Gerrit-Change-Number: 11970
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>

Reply via email to