Adar Dembo has posted comments on this change.

Change subject: thirdparty: add boost and switch to header-only build

Patch Set 1:

Commit Message:

Line 18:    libstdc++, and libsasl, all of which should not be managed due to 
> and openssl, although the point still stands.
Right, though I explicitly omitted openssl because, at least for now, you 
really can't configure Kudu to use it.

PS1, Line 22: Some distros (e.g. SLES12) don't offer
            :    system packages
> The specific issue with SLES 12 (and I would assume the majority of the sma
Ah, I misread the blurb in installation.adoc. Will update.
File docs/contributing.adoc:

Line 151: We are in the process of removing all remaining `boost` dependencies 
from the
> We still have 'approved' boost libraries.  I would vote to keep the approve
I really was hoping to avoid introducing new Boost usage, so that we stand a 
chance of removing the whole thing some day.

Is your point that adding a new usage of e.g. boost::optional isn't a big deal 
because, if/when we do replace it, updating an additional call-site is no big 

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: Id5b73a8e2a86099429b6032023a01a0da7b02371
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <>
Gerrit-Reviewer: Adar Dembo <>
Gerrit-Reviewer: Dan Burkert <>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <>
Gerrit-HasComments: Yes

Reply via email to