Yingchun Lai has posted comments on this change. ( http://gerrit.cloudera.org:8080/21075 )
Change subject: KUDU-3371 [fs] Use RocksDB to store LBM metadata (2nd try) ...................................................................... Patch Set 6: > Patch Set 6: > > Just a clarifying question, so the only thing changed compared to the 1st > revision of this patch is that lz4 is switched out for snappy? -> only > changes in thirdparty/build-definitions.sh ? > > Thanks for debugging the issue! You can review the changes by https://gerrit.cloudera.org/c/21075/1..6, the changed files are: Commit message cmake_modules/FindSnappy.cmake src/kudu/fs/CMakeLists.txt src/kudu/server/CMakeLists.txt src/kudu/util/CMakeLists.txt thirdparty/build-definitions.sh -- To view, visit http://gerrit.cloudera.org:8080/21075 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I23b7d2a16802af01a382a1d74cd9869baf364688 Gerrit-Change-Number: 21075 Gerrit-PatchSet: 6 Gerrit-Owner: Yingchun Lai <laiyingc...@apache.org> Gerrit-Reviewer: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <greber...@gmail.com> Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Yingchun Lai <laiyingc...@apache.org> Gerrit-Comment-Date: Wed, 20 Mar 2024 02:50:00 +0000 Gerrit-HasComments: No