PragmaTwice commented on code in PR #2688:
URL: https://github.com/apache/kvrocks/pull/2688#discussion_r1872621095
##########
src/config/config.cc:
##########
@@ -131,8 +131,8 @@ Status SetRocksdbCompression(Server *srv, const
rocksdb::CompressionType compres
for (size_t i = compression_start_level; i < KVROCKS_MAX_LSM_LEVEL; i++) {
compression_per_level_builder.emplace_back(compression_option);
}
- const std::string compression_per_level = util::StringJoin(
- compression_per_level_builder, [](const auto &s) -> decltype(auto) {
return s; }, ":");
+ const std::string compression_per_level =
+ util::StringJoin(compression_per_level_builder, [](const auto &s) ->
decltype(auto) { return s; }, ":");
Review Comment:
Seems we need to revert this change to make CI pass.
Note that in the CI the clang-format version is 14. Different versions may
lead to different format results.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
