Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15010 )
Change subject: fs: remove test-only constructor ...................................................................... Patch Set 1: It seems there is one place which required the constructor just removed: 17:27:41 /home/jenkins-slave/workspace/kudu-master/0/src/kudu/tablet/compaction-test.cc:445:17: error: no matching constructor for initialization of 'kudu::FsManager' 17:27:41 FsManager fs_manager(env_, FLAGS_merge_benchmark_input_dir); -- To view, visit http://gerrit.cloudera.org:8080/15010 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I11024a2f28123bba3e3989ddd5e68f581481d5fb Gerrit-Change-Number: 15010 Gerrit-PatchSet: 1 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Sat, 11 Jan 2020 04:30:48 +0000 Gerrit-HasComments: No
