Adar Dembo has posted comments on this change. Change subject: Replace uses of boost::thread with std::thread ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/1868/1//COMMIT_MSG Commit Message: Line 7: Replace uses of boost::thread with std::thread > I agree in the abstract, but let's not scope creep here, and I dont think i Okay, we can defer the kudu::thread transition to later, but then what's the point of switching to std::thread in the meantime? The codebase will now have three kinds of threads instead of two. What do we gain? I don't think we lose the boost_thread library dependency yet, or do we? -- To view, visit http://gerrit.cloudera.org:8080/1868 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I27e14c505cbfacf791f9730dc343196ea720134b Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
