Re: Review Request 58448: Windows: Updated ZooKeeper to use CMake.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58448/#review173018 --- Ship it! The trailing whitespace in the patch breaks our linter, but since all those whitespaces are in the ZK source, there's nothing we can do about it :( - Joseph Wu On April 19, 2017, 11:20 a.m., Andrew Schwartzmeyer wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58448/ > --- > > (Updated April 19, 2017, 11:20 a.m.) > > > Review request for mesos, Jeff Coffler, John Kordich, Joseph Wu, and Li Li. > > > Bugs: ZOOKEEPER-2756 > https://issues.apache.org/jira/browse/ZOOKEEPER-2756 > > > Repository: mesos > > > Description > --- > > This unblocks us from building exclusively with VS 2017. The previous > patch to ZooKeeper only added VS 2015 support. This patch replaces it > with a CMake build system that will generate whichever solution we need > for Windows (and can replace the Autotools system on Linux). > > We're updating to 3.5.2-alpha as the existing 3.5.1 rebundle was a > source tarball, and so missing the necessary generated files. The most > currently used version was based off a random commit. 3.5.2-alpha is the > latest 3.5.x release of ZooKeeper (3.5.x itself is alpha, 3.5.2 is > semi-stable, in comparison to 3.5.3 which is in RC). > > > Diffs > - > > 3rdparty/CMakeLists.txt bb61ef0514fb164f35b34bb6be1bbebb4d1a1861 > 3rdparty/cmake/Mesos3rdpartyConfigure.cmake > c60652688a23f8628f133b7890ff39e38fc8ae94 > 3rdparty/cmake/Versions.cmake 912726351ff744dd839b8d1c8d64dcc373d879be > 3rdparty/zookeeper-3.5.2-alpha.patch PRE-CREATION > > > Diff: https://reviews.apache.org/r/58448/diff/1/ > > > Testing > --- > > cmake and make check on Linux > stout-tests, libprocess-tests, and mesos-tests on Windows: all tests passed. > (Note: my machine updated to the Creators Update today, disabling long path > support; the previous way to enable it doesn't seem to work, so I re-ran the > failed tests on a machine that hadn't updated and still had long path > support, they passed). > > Also tested the build on a clean machine with nothing but the VS 2017 build > tools (no IDE, no prior VS installations); mesos-tests builds no problem. > > > Thanks, > > Andrew Schwartzmeyer > >
Re: Review Request 58448: Windows: Updated ZooKeeper to use CMake.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58448/#review172809 --- Ship it! Ship It! - Jeff Coffler On April 19, 2017, 6:20 p.m., Andrew Schwartzmeyer wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58448/ > --- > > (Updated April 19, 2017, 6:20 p.m.) > > > Review request for mesos, Jeff Coffler, John Kordich, Joseph Wu, and Li Li. > > > Bugs: ZOOKEEPER-2756 > https://issues.apache.org/jira/browse/ZOOKEEPER-2756 > > > Repository: mesos > > > Description > --- > > This unblocks us from building exclusively with VS 2017. The previous > patch to ZooKeeper only added VS 2015 support. This patch replaces it > with a CMake build system that will generate whichever solution we need > for Windows (and can replace the Autotools system on Linux). > > We're updating to 3.5.2-alpha as the existing 3.5.1 rebundle was a > source tarball, and so missing the necessary generated files. The most > currently used version was based off a random commit. 3.5.2-alpha is the > latest 3.5.x release of ZooKeeper (3.5.x itself is alpha, 3.5.2 is > semi-stable, in comparison to 3.5.3 which is in RC). > > > Diffs > - > > 3rdparty/CMakeLists.txt bb61ef0514fb164f35b34bb6be1bbebb4d1a1861 > 3rdparty/cmake/Mesos3rdpartyConfigure.cmake > c60652688a23f8628f133b7890ff39e38fc8ae94 > 3rdparty/cmake/Versions.cmake 912726351ff744dd839b8d1c8d64dcc373d879be > 3rdparty/zookeeper-3.5.2-alpha.patch PRE-CREATION > > > Diff: https://reviews.apache.org/r/58448/diff/1/ > > > Testing > --- > > cmake and make check on Linux > stout-tests, libprocess-tests, and mesos-tests on Windows: all tests passed. > (Note: my machine updated to the Creators Update today, disabling long path > support; the previous way to enable it doesn't seem to work, so I re-ran the > failed tests on a machine that hadn't updated and still had long path > support, they passed). > > Also tested the build on a clean machine with nothing but the VS 2017 build > tools (no IDE, no prior VS installations); mesos-tests builds no problem. > > > Thanks, > > Andrew Schwartzmeyer > >
Re: Review Request 58448: Windows: Updated ZooKeeper to use CMake.
> On April 18, 2017, 8:30 p.m., Jeff Coffler wrote: > > Big change, but important for us. > > > > I'd like you to comment on using cmake for zookeeper, specifically in terms > > of rolling that back to Zookeeper themselves for them to maintain it. > > > > Have you looked at contributing this to the Zookeeper folks for them to use > > and maintain? What was their response? > > > > This feels like something that is easily broken (mentioning specific > > sources, etc) if they aren't maintaining it, and I'd rather not own this > > for life. If the Zookeeper folks don't want to maintain this, what do they > > say about VS 2017 support? Even if it's not cmake, but a hand-generated > > solution for V/S, at least they maintain it, not us. > > Andrew Schwartzmeyer wrote: > Ah, sorry, I moved bugs around. Just attached the associated ZooKeeper > bug with what you're looking for. > > Jeff Coffler wrote: > But my question remains: Will the Zookeeper folks take this and own it, > or is it ours for live? More importantly, what do the Zookeeper folks have to > say about VS 2017 support? Do they have an answer, or are they embracing your > change as the answer? > > This really comes down to: Who owns this with time? Us (if we take a new > version of Zookeeper), or the Zookeeper team? They are [reviewing the patch](https://issues.apache.org/jira/browse/ZOOKEEPER-2756) and have requested a PR over a patch file. We own our patches, this simply replaces the current patches to ZooKeeper we already own. Theoretically we could update to a newer ZooKeeper if they release one with my changes included, but it doesn't block us. - Andrew --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58448/#review172263 --- On April 19, 2017, 6:20 p.m., Andrew Schwartzmeyer wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58448/ > --- > > (Updated April 19, 2017, 6:20 p.m.) > > > Review request for mesos, Jeff Coffler, John Kordich, Joseph Wu, and Li Li. > > > Bugs: ZOOKEEPER-2756 > https://issues.apache.org/jira/browse/ZOOKEEPER-2756 > > > Repository: mesos > > > Description > --- > > This unblocks us from building exclusively with VS 2017. The previous > patch to ZooKeeper only added VS 2015 support. This patch replaces it > with a CMake build system that will generate whichever solution we need > for Windows (and can replace the Autotools system on Linux). > > We're updating to 3.5.2-alpha as the existing 3.5.1 rebundle was a > source tarball, and so missing the necessary generated files. The most > currently used version was based off a random commit. 3.5.2-alpha is the > latest 3.5.x release of ZooKeeper (3.5.x itself is alpha, 3.5.2 is > semi-stable, in comparison to 3.5.3 which is in RC). > > > Diffs > - > > 3rdparty/CMakeLists.txt bb61ef0514fb164f35b34bb6be1bbebb4d1a1861 > 3rdparty/cmake/Mesos3rdpartyConfigure.cmake > c60652688a23f8628f133b7890ff39e38fc8ae94 > 3rdparty/cmake/Versions.cmake 912726351ff744dd839b8d1c8d64dcc373d879be > 3rdparty/zookeeper-3.5.2-alpha.patch PRE-CREATION > > > Diff: https://reviews.apache.org/r/58448/diff/1/ > > > Testing > --- > > cmake and make check on Linux > stout-tests, libprocess-tests, and mesos-tests on Windows: all tests passed. > (Note: my machine updated to the Creators Update today, disabling long path > support; the previous way to enable it doesn't seem to work, so I re-ran the > failed tests on a machine that hadn't updated and still had long path > support, they passed). > > Also tested the build on a clean machine with nothing but the VS 2017 build > tools (no IDE, no prior VS installations); mesos-tests builds no problem. > > > Thanks, > > Andrew Schwartzmeyer > >
Re: Review Request 58448: Windows: Updated ZooKeeper to use CMake.
> On April 18, 2017, 8:30 p.m., Jeff Coffler wrote: > > Big change, but important for us. > > > > I'd like you to comment on using cmake for zookeeper, specifically in terms > > of rolling that back to Zookeeper themselves for them to maintain it. > > > > Have you looked at contributing this to the Zookeeper folks for them to use > > and maintain? What was their response? > > > > This feels like something that is easily broken (mentioning specific > > sources, etc) if they aren't maintaining it, and I'd rather not own this > > for life. If the Zookeeper folks don't want to maintain this, what do they > > say about VS 2017 support? Even if it's not cmake, but a hand-generated > > solution for V/S, at least they maintain it, not us. > > Andrew Schwartzmeyer wrote: > Ah, sorry, I moved bugs around. Just attached the associated ZooKeeper > bug with what you're looking for. But my question remains: Will the Zookeeper folks take this and own it, or is it ours for live? More importantly, what do the Zookeeper folks have to say about VS 2017 support? Do they have an answer, or are they embracing your change as the answer? This really comes down to: Who owns this with time? Us (if we take a new version of Zookeeper), or the Zookeeper team? - Jeff --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58448/#review172263 --- On April 19, 2017, 6:20 p.m., Andrew Schwartzmeyer wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58448/ > --- > > (Updated April 19, 2017, 6:20 p.m.) > > > Review request for mesos, Jeff Coffler, John Kordich, Joseph Wu, and Li Li. > > > Bugs: ZOOKEEPER-2756 > https://issues.apache.org/jira/browse/ZOOKEEPER-2756 > > > Repository: mesos > > > Description > --- > > This unblocks us from building exclusively with VS 2017. The previous > patch to ZooKeeper only added VS 2015 support. This patch replaces it > with a CMake build system that will generate whichever solution we need > for Windows (and can replace the Autotools system on Linux). > > We're updating to 3.5.2-alpha as the existing 3.5.1 rebundle was a > source tarball, and so missing the necessary generated files. The most > currently used version was based off a random commit. 3.5.2-alpha is the > latest 3.5.x release of ZooKeeper (3.5.x itself is alpha, 3.5.2 is > semi-stable, in comparison to 3.5.3 which is in RC). > > > Diffs > - > > 3rdparty/CMakeLists.txt bb61ef0514fb164f35b34bb6be1bbebb4d1a1861 > 3rdparty/cmake/Mesos3rdpartyConfigure.cmake > c60652688a23f8628f133b7890ff39e38fc8ae94 > 3rdparty/cmake/Versions.cmake 912726351ff744dd839b8d1c8d64dcc373d879be > 3rdparty/zookeeper-3.5.2-alpha.patch PRE-CREATION > > > Diff: https://reviews.apache.org/r/58448/diff/1/ > > > Testing > --- > > cmake and make check on Linux > stout-tests, libprocess-tests, and mesos-tests on Windows: all tests passed. > (Note: my machine updated to the Creators Update today, disabling long path > support; the previous way to enable it doesn't seem to work, so I re-ran the > failed tests on a machine that hadn't updated and still had long path > support, they passed). > > Also tested the build on a clean machine with nothing but the VS 2017 build > tools (no IDE, no prior VS installations); mesos-tests builds no problem. > > > Thanks, > > Andrew Schwartzmeyer > >
Re: Review Request 58448: Windows: Updated ZooKeeper to use CMake.
> On April 18, 2017, 8:30 p.m., Jeff Coffler wrote: > > Big change, but important for us. > > > > I'd like you to comment on using cmake for zookeeper, specifically in terms > > of rolling that back to Zookeeper themselves for them to maintain it. > > > > Have you looked at contributing this to the Zookeeper folks for them to use > > and maintain? What was their response? > > > > This feels like something that is easily broken (mentioning specific > > sources, etc) if they aren't maintaining it, and I'd rather not own this > > for life. If the Zookeeper folks don't want to maintain this, what do they > > say about VS 2017 support? Even if it's not cmake, but a hand-generated > > solution for V/S, at least they maintain it, not us. Ah, sorry, I moved bugs around. Just attached the associated ZooKeeper bug with what you're looking for. - Andrew --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58448/#review172263 --- On April 19, 2017, 6:20 p.m., Andrew Schwartzmeyer wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58448/ > --- > > (Updated April 19, 2017, 6:20 p.m.) > > > Review request for mesos, Jeff Coffler, John Kordich, Joseph Wu, and Li Li. > > > Bugs: ZOOKEEPER-2756 > https://issues.apache.org/jira/browse/ZOOKEEPER-2756 > > > Repository: mesos > > > Description > --- > > This unblocks us from building exclusively with VS 2017. The previous > patch to ZooKeeper only added VS 2015 support. This patch replaces it > with a CMake build system that will generate whichever solution we need > for Windows (and can replace the Autotools system on Linux). > > We're updating to 3.5.2-alpha as the existing 3.5.1 rebundle was a > source tarball, and so missing the necessary generated files. The most > currently used version was based off a random commit. 3.5.2-alpha is the > latest 3.5.x release of ZooKeeper (3.5.x itself is alpha, 3.5.2 is > semi-stable, in comparison to 3.5.3 which is in RC). > > > Diffs > - > > 3rdparty/CMakeLists.txt bb61ef0514fb164f35b34bb6be1bbebb4d1a1861 > 3rdparty/cmake/Mesos3rdpartyConfigure.cmake > c60652688a23f8628f133b7890ff39e38fc8ae94 > 3rdparty/cmake/Versions.cmake 912726351ff744dd839b8d1c8d64dcc373d879be > 3rdparty/zookeeper-3.5.2-alpha.patch PRE-CREATION > > > Diff: https://reviews.apache.org/r/58448/diff/1/ > > > Testing > --- > > cmake and make check on Linux > stout-tests, libprocess-tests, and mesos-tests on Windows: all tests passed. > (Note: my machine updated to the Creators Update today, disabling long path > support; the previous way to enable it doesn't seem to work, so I re-ran the > failed tests on a machine that hadn't updated and still had long path > support, they passed). > > Also tested the build on a clean machine with nothing but the VS 2017 build > tools (no IDE, no prior VS installations); mesos-tests builds no problem. > > > Thanks, > > Andrew Schwartzmeyer > >
Re: Review Request 58448: Windows: Updated ZooKeeper to use CMake.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58448/ --- (Updated April 19, 2017, 6:20 p.m.) Review request for mesos, Jeff Coffler, John Kordich, Joseph Wu, and Li Li. Changes --- Add ZooKeeper bug. Bugs: ZOOKEEPER-2756 https://issues.apache.org/jira/browse/ZOOKEEPER-2756 Repository: mesos Description --- This unblocks us from building exclusively with VS 2017. The previous patch to ZooKeeper only added VS 2015 support. This patch replaces it with a CMake build system that will generate whichever solution we need for Windows (and can replace the Autotools system on Linux). We're updating to 3.5.2-alpha as the existing 3.5.1 rebundle was a source tarball, and so missing the necessary generated files. The most currently used version was based off a random commit. 3.5.2-alpha is the latest 3.5.x release of ZooKeeper (3.5.x itself is alpha, 3.5.2 is semi-stable, in comparison to 3.5.3 which is in RC). Diffs - 3rdparty/CMakeLists.txt bb61ef0514fb164f35b34bb6be1bbebb4d1a1861 3rdparty/cmake/Mesos3rdpartyConfigure.cmake c60652688a23f8628f133b7890ff39e38fc8ae94 3rdparty/cmake/Versions.cmake 912726351ff744dd839b8d1c8d64dcc373d879be 3rdparty/zookeeper-3.5.2-alpha.patch PRE-CREATION Diff: https://reviews.apache.org/r/58448/diff/1/ Testing --- cmake and make check on Linux stout-tests, libprocess-tests, and mesos-tests on Windows: all tests passed. (Note: my machine updated to the Creators Update today, disabling long path support; the previous way to enable it doesn't seem to work, so I re-ran the failed tests on a machine that hadn't updated and still had long path support, they passed). Also tested the build on a clean machine with nothing but the VS 2017 build tools (no IDE, no prior VS installations); mesos-tests builds no problem. Thanks, Andrew Schwartzmeyer
Re: Review Request 58448: Windows: Updated ZooKeeper to use CMake.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58448/#review172263 --- Big change, but important for us. I'd like you to comment on using cmake for zookeeper, specifically in terms of rolling that back to Zookeeper themselves for them to maintain it. Have you looked at contributing this to the Zookeeper folks for them to use and maintain? What was their response? This feels like something that is easily broken (mentioning specific sources, etc) if they aren't maintaining it, and I'd rather not own this for life. If the Zookeeper folks don't want to maintain this, what do they say about VS 2017 support? Even if it's not cmake, but a hand-generated solution for V/S, at least they maintain it, not us. - Jeff Coffler On April 14, 2017, 4:12 a.m., Andrew Schwartzmeyer wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58448/ > --- > > (Updated April 14, 2017, 4:12 a.m.) > > > Review request for mesos, Jeff Coffler, John Kordich, Joseph Wu, and Li Li. > > > Repository: mesos > > > Description > --- > > This unblocks us from building exclusively with VS 2017. The previous > patch to ZooKeeper only added VS 2015 support. This patch replaces it > with a CMake build system that will generate whichever solution we need > for Windows (and can replace the Autotools system on Linux). > > We're updating to 3.5.2-alpha as the existing 3.5.1 rebundle was a > source tarball, and so missing the necessary generated files. The most > currently used version was based off a random commit. 3.5.2-alpha is the > latest 3.5.x release of ZooKeeper (3.5.x itself is alpha, 3.5.2 is > semi-stable, in comparison to 3.5.3 which is in RC). > > > Diffs > - > > 3rdparty/CMakeLists.txt bb61ef0514fb164f35b34bb6be1bbebb4d1a1861 > 3rdparty/cmake/Mesos3rdpartyConfigure.cmake > c60652688a23f8628f133b7890ff39e38fc8ae94 > 3rdparty/cmake/Versions.cmake 912726351ff744dd839b8d1c8d64dcc373d879be > 3rdparty/zookeeper-3.5.2-alpha.patch PRE-CREATION > > > Diff: https://reviews.apache.org/r/58448/diff/1/ > > > Testing > --- > > cmake and make check on Linux > stout-tests, libprocess-tests, and mesos-tests on Windows: all tests passed. > (Note: my machine updated to the Creators Update today, disabling long path > support; the previous way to enable it doesn't seem to work, so I re-ran the > failed tests on a machine that hadn't updated and still had long path > support, they passed). > > Also tested the build on a clean machine with nothing but the VS 2017 build > tools (no IDE, no prior VS installations); mesos-tests builds no problem. > > > Thanks, > > Andrew Schwartzmeyer > >
Re: Review Request 58448: Windows: Updated ZooKeeper to use CMake.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58448/ --- (Updated April 14, 2017, 4:12 a.m.) Review request for mesos, Jeff Coffler, John Kordich, Joseph Wu, and Li Li. Repository: mesos Description --- This unblocks us from building exclusively with VS 2017. The previous patch to ZooKeeper only added VS 2015 support. This patch replaces it with a CMake build system that will generate whichever solution we need for Windows (and can replace the Autotools system on Linux). We're updating to 3.5.2-alpha as the existing 3.5.1 rebundle was a source tarball, and so missing the necessary generated files. The most currently used version was based off a random commit. 3.5.2-alpha is the latest 3.5.x release of ZooKeeper (3.5.x itself is alpha, 3.5.2 is semi-stable, in comparison to 3.5.3 which is in RC). Diffs - 3rdparty/CMakeLists.txt bb61ef0514fb164f35b34bb6be1bbebb4d1a1861 3rdparty/cmake/Mesos3rdpartyConfigure.cmake c60652688a23f8628f133b7890ff39e38fc8ae94 3rdparty/cmake/Versions.cmake 912726351ff744dd839b8d1c8d64dcc373d879be 3rdparty/zookeeper-3.5.2-alpha.patch PRE-CREATION Diff: https://reviews.apache.org/r/58448/diff/1/ Testing (updated) --- cmake and make check on Linux stout-tests, libprocess-tests, and mesos-tests on Windows: all tests passed. (Note: my machine updated to the Creators Update today, disabling long path support; the previous way to enable it doesn't seem to work, so I re-ran the failed tests on a machine that hadn't updated and still had long path support, they passed). Also tested the build on a clean machine with nothing but the VS 2017 build tools (no IDE, no prior VS installations); mesos-tests builds no problem. Thanks, Andrew Schwartzmeyer
Re: Review Request 58448: Windows: Updated ZooKeeper to use CMake.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58448/#review171981 --- Please note that this patch was also [submitted to ZooKeeper](https://issues.apache.org/jira/browse/ZOOKEEPER-2756), though it's final form for the upstream project will probably be different (rebased onto master and split into multiple patches). - Andrew Schwartzmeyer On April 14, 2017, 2:15 a.m., Andrew Schwartzmeyer wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58448/ > --- > > (Updated April 14, 2017, 2:15 a.m.) > > > Review request for mesos, Jeff Coffler, John Kordich, Joseph Wu, and Li Li. > > > Repository: mesos > > > Description > --- > > This unblocks us from building exclusively with VS 2017. The previous > patch to ZooKeeper only added VS 2015 support. This patch replaces it > with a CMake build system that will generate whichever solution we need > for Windows (and can replace the Autotools system on Linux). > > We're updating to 3.5.2-alpha as the existing 3.5.1 rebundle was a > source tarball, and so missing the necessary generated files. The most > currently used version was based off a random commit. 3.5.2-alpha is the > latest 3.5.x release of ZooKeeper (3.5.x itself is alpha, 3.5.2 is > semi-stable, in comparison to 3.5.3 which is in RC). > > > Diffs > - > > 3rdparty/CMakeLists.txt bb61ef0514fb164f35b34bb6be1bbebb4d1a1861 > 3rdparty/cmake/Mesos3rdpartyConfigure.cmake > c60652688a23f8628f133b7890ff39e38fc8ae94 > 3rdparty/cmake/Versions.cmake 912726351ff744dd839b8d1c8d64dcc373d879be > 3rdparty/zookeeper-3.5.2-alpha.patch PRE-CREATION > > > Diff: https://reviews.apache.org/r/58448/diff/1/ > > > Testing > --- > > cmake and make check on Linux > stout-tests, libprocess-tests, and mesos-tests on Windows (last still > pending, they're so slow). > > Also testing the build on a clean machine with nothing but the VS 2017 build > tools (no IDE, no prior VS installations), results pending. > > > Thanks, > > Andrew Schwartzmeyer > >
Re: Review Request 58448: Windows: Updated ZooKeeper to use CMake.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58448/ --- (Updated April 14, 2017, 2:15 a.m.) Review request for mesos, Jeff Coffler, John Kordich, Joseph Wu, and Li Li. Repository: mesos Description --- This unblocks us from building exclusively with VS 2017. The previous patch to ZooKeeper only added VS 2015 support. This patch replaces it with a CMake build system that will generate whichever solution we need for Windows (and can replace the Autotools system on Linux). We're updating to 3.5.2-alpha as the existing 3.5.1 rebundle was a source tarball, and so missing the necessary generated files. The most currently used version was based off a random commit. 3.5.2-alpha is the latest 3.5.x release of ZooKeeper (3.5.x itself is alpha, 3.5.2 is semi-stable, in comparison to 3.5.3 which is in RC). Diffs - 3rdparty/CMakeLists.txt bb61ef0514fb164f35b34bb6be1bbebb4d1a1861 3rdparty/cmake/Mesos3rdpartyConfigure.cmake c60652688a23f8628f133b7890ff39e38fc8ae94 3rdparty/cmake/Versions.cmake 912726351ff744dd839b8d1c8d64dcc373d879be 3rdparty/zookeeper-3.5.2-alpha.patch PRE-CREATION Diff: https://reviews.apache.org/r/58448/diff/1/ Testing (updated) --- cmake and make check on Linux stout-tests, libprocess-tests, and mesos-tests on Windows (last still pending, they're so slow). Also testing the build on a clean machine with nothing but the VS 2017 build tools (no IDE, no prior VS installations), results pending. Thanks, Andrew Schwartzmeyer