Hello Mike Percy, Adar Dembo, I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/3951 to review the following change. Change subject: KUDU-1555. PBC Flush() method should be async ...................................................................... KUDU-1555. PBC Flush() method should be async WritablePBContainerFile::Flush is meant to flush the data asynchronously. But, e6052ac accidentally regressed this functionality and switched it to synchronous. This caused a lack of parallelism in the LogBlockManager flush path, since it uses PBC files for the metadata. The async flushing of LBM data blocks ended up being basically synchronous, so each disk would have to wait for the prior disk to complete its flush before it started any IO. I tested this change on a YCSB workload and throughput increased almost 2x. Change-Id: I721707070fe47e3377d791c95214f007c90d2263 --- M src/kudu/util/pb_util.cc 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/51/3951/1 -- To view, visit http://gerrit.cloudera.org:8080/3951 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I721707070fe47e3377d791c95214f007c90d2263 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Mike Percy <mpe...@apache.org>