Yingchun Lai has posted comments on this change. ( http://gerrit.cloudera.org:8080/14743 )
Change subject: KUDU-3001 Multi-thread to load containers in a data directory ...................................................................... Patch Set 13: (2 comments) http://gerrit.cloudera.org:8080/#/c/14743/12//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/14743/12//COMMIT_MSG@32 PS12, Line 32: | new version > Interesting. Your results suggest that >1 threads on spinning disks are ben Done http://gerrit.cloudera.org:8080/#/c/14743/12/src/kudu/fs/log_block_manager.h File src/kudu/fs/log_block_manager.h: http://gerrit.cloudera.org:8080/#/c/14743/12/src/kudu/fs/log_block_manager.h@261 PS12, Line 261: // Adds an as of yet unseen container to this block manager. > This is only used in one place now; could we remove it to avoid confusion w Done -- To view, visit http://gerrit.cloudera.org:8080/14743 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I0721ee4a5a6824db146ba0658e60eec25dd0c65c Gerrit-Change-Number: 14743 Gerrit-PatchSet: 13 Gerrit-Owner: Yingchun Lai <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Reviewer: wangning <[email protected]> Gerrit-Comment-Date: Fri, 06 Dec 2019 11:36:49 +0000 Gerrit-HasComments: Yes
