Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9896 )
Change subject: pprof: fix contention output ...................................................................... pprof: fix contention output In 600723e0e954a1b199e6a0bf1dfd63fe7289a6a8 I changed the profile to be written to an intermediary buffer before writing to the eventual HTTP output stream, but missed one location where it was still writing directly. This caused some output that would look like: <profile samples> <header> <more profile samples> ...which confused the pprof tool. Change-Id: I6b55a47c3e23daa62302b4318580592695ec0f5f Reviewed-on: http://gerrit.cloudera.org:8080/9896 Tested-by: Kudu Jenkins Reviewed-by: Andrew Wong <[email protected]> --- M src/kudu/server/pprof_path_handlers.cc 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Kudu Jenkins: Verified Andrew Wong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/9896 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I6b55a47c3e23daa62302b4318580592695ec0f5f Gerrit-Change-Number: 9896 Gerrit-PatchSet: 3 Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
