{"push_id":793432809,"size":67,"distinct_size":67,"ref":"refs/heads/executor_proto","head":"601c9d95cbec1342bce19b121cd433b2d2f4e959","before":"a3e83c82e8474ac5c7b5e4d8a541f841fe107cfc","commits":[{"sha":"3446c25ca9450807c2547d829412162282e38038","author":{"email":"2e6147cbff6661bd424c256de746b4f230002dff@gmail.com","name":"Alex Clemmer"},"message":"Changed build commands for 3rdparty libs to be no-ops in Windows.\n\nLargely a reverse of a previous commit. We previously removed the quotes\nfrom around some interpolated variables, to instead make them regular\nvariable expansions, but in their context (they expand to build commands\nfor third-party libraries) we now know that this doesn't work on all\nplatforms for all versions of CMake.\n\nA longer explanation of the technical issue is in the code comments.\n\nReview: https://reviews.apache.org/r/37019","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/3446c25ca9450807c2547d829412162282e38038"},{"sha":"66b6ab84091428f2f3391887e29fbfe9ad14b64f","author":{"email":"2e6147cbff6661bd424c256de746b4f230002dff@gmail.com","name":"Alex Clemmer"},"message":"Transitioned Windows builds to use glog v0.3.4.\n\nTo get Stout tests to compile on Windows, it is necessary to upgrade the\nglog dependency to 0.3.4 (which will allow you to compile with MSVC\n1800), and then port glog 0.3.4 to support compiling with MSVC 1900\n(which Mesos requires). The code comments explain why we can't compile\nwith 1800 and link simply have Mesos link to it.\n\nNote that we have supplied the MSVC 1900 port to the glog project, under\npull request #43[1].\n\nThe longer story follows:\n\nglog v0.3.3 will not build on Windows for a few reasons that are fixed\nin the 0.3.4 release. (For example, `algorithms` is not included in one\nfile the requires it.)\n\nSince 0.3.4 appears to only fix bugs, and because there appear to be no\nmajor changes in (e.g.) the ABI, I believe this is a safe change to\nmake. Additionally, when we ported it to compile on MSVC 1900, we didn't\nwant to re-write the changes they made in 0.3.4 to make it run on MSVC\n1800, so this was a natural starting point for us.\n\nI choose not to attempt to upgrade the Linux glog dependency, either in\nthe CMake solution, or in the autotools solution, for 3 reasons.\n\n(1) I don't want to ask the community reconsider its dependency choices\n    on the basis that an experimental Windows port requires a more\n    recent version.\n(2) We are currently rebundling the glog library as a binary, which\n    means that a solution will be somewhat involved (i.e., either we\n    have to change the patch command we're running on glog, or we have\n    to change the binary)\n(3) We can easily support both versions in CMake using the very powerful\n    macro system, which will allow us download the dependency on Windows\n    machines and use the tarball on other systems.\n\nFootnotes:\n[1] https://github.com/google/glog/pull/43\n\nReview: https://reviews.apache.org/r/37020","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/66b6ab84091428f2f3391887e29fbfe9ad14b64f"},{"sha":"069f2e3b9ae28b54b6fd550f65662022726c7b73","author":{"email":"2e6147cbff6661bd424c256de746b4f230002dff@gmail.com","name":"Alex Clemmer"},"message":"Removed glog signal handler installation on Windows.\n\nglog declares, but does not define the function\n`google::InstallFailureSignalHandler`. If you try to compile a call to\nthis function, you will get a linker error.\n\nSince we can't use it, this commit will remove that code on Windows.\n\nReview: https://reviews.apache.org/r/37021","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/069f2e3b9ae28b54b6fd550f65662022726c7b73"},{"sha":"92ed652bf179fb93f9040602c2e8d74a90bb7114","author":{"email":"2e6147cbff6661bd424c256de746b4f230002dff@gmail.com","name":"Alex Clemmer"},"message":"Modified definitions in windows.hpp to fit real compilation problems.\n\nEssentially, there are some definitions that need to be there, and some\nthat are not quite right.\n\nThis commit adds and removes the definitions required to get the ip\ntests to pass. We also add some nice comments.\n\nReview: https://reviews.apache.org/r/37030","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/92ed652bf179fb93f9040602c2e8d74a90bb7114"},{"sha":"342f2b301ef42dee3077351db8debdc054e32848","author":{"email":"2e6147cbff6661bd424c256de746b4f230002dff@gmail.com","name":"Alex Clemmer"},"message":"Implemented os::close for Windows.\n\nReview: https://reviews.apache.org/r/37031","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/342f2b301ef42dee3077351db8debdc054e32848"},{"sha":"3b03427790a9d443c677bbbebd6016b798302b1e","author":{"email":"2e6147cbff6661bd424c256de746b4f230002dff@gmail.com","name":"Alex Clemmer"},"message":"Extended permissions.hpp to work on both Windows and POSIX.\n\nReview: https://reviews.apache.org/r/37032","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/3b03427790a9d443c677bbbebd6016b798302b1e"},{"sha":"989eca4db7301f8cdae37cc9f801095ff0061929","author":{"email":"2e6147cbff6661bd424c256de746b4f230002dff@gmail.com","name":"Alex Clemmer"},"message":"Implemented stout/os/windows/exists.hpp.\n\nReview: https://reviews.apache.org/r/37344","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/989eca4db7301f8cdae37cc9f801095ff0061929"},{"sha":"066a5a8000b98199aa799c53def7ccbff76a82a6","author":{"email":"2e6147cbff6661bd424c256de746b4f230002dff@gmail.com","name":"Alex Clemmer"},"message":"Modernized the Stout tests CMakeLists file.\n\nNow that other people are starting to attempt to compile Stout on\nWindows, it's important that we give them a CMakeLists file that\nexplicitly compiles the parts of Stout we're willing to support.\n\nReview: https://reviews.apache.org/r/37345","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/066a5a8000b98199aa799c53def7ccbff76a82a6"},{"sha":"788580cd808c998b6ae472c27bf8728ca37bf280","author":{"email":"2e6147cbff6661bd424c256de746b4f230002dff@gmail.com","name":"Alex Clemmer"},"message":"Cleaned up stout/windows.hpp.\n\nReview: https://reviews.apache.org/r/37368","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/788580cd808c998b6ae472c27bf8728ca37bf280"},{"sha":"588b94e7806b4fb5ad2abb02b6229405858eef76","author":{"email":"2e6147cbff6661bd424c256de746b4f230002dff@gmail.com","name":"Alex Clemmer"},"message":"Added firewall.cpp to the process library CMakeLists.\n\nRight now, if you try to build on a non-Windows machine with CMakeLists,\nyou'll get a linker error claiming that we can't find a symbol from the\nprocess library's firewall.cpp.\n\nThis commit simply adds that source file, which fixes the problem.\n\nReview: https://reviews.apache.org/r/37369","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/588b94e7806b4fb5ad2abb02b6229405858eef76"},{"sha":"cc0b6ea87f2ca9bb2d51d18f189b8d1bccc1c7a1","author":{"email":"2e6147cbff6661bd424c256de746b4f230002dff@gmail.com","name":"Alex Clemmer"},"message":"Fixed CMake build compile error; don't compile GMock with cxx11 flag.\n\nRight now if you try to compile GMock a non-Windows platform using the\nCMake build system, it will error out. The problem seems to be the CXX11\nflag.\n\nThis commit will solve this problem by removing that flag.\n\nReview: https://reviews.apache.org/r/37370","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/cc0b6ea87f2ca9bb2d51d18f189b8d1bccc1c7a1"},{"sha":"81efd727eb3087378c33a84cc3fd315f31f2edc5","author":{"email":"0c029e190c301746fa0adb35fa5dc69edca45ca3@gmail.com","name":"Michael Park"},"message":"Fixed flaky ReservationEndpointsTest.AvailableResources test.\n\nReview: https://reviews.apache.org/r/38282","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/81efd727eb3087378c33a84cc3fd315f31f2edc5"},{"sha":"941174914d309477b05865078ecf8a0f69db04f8","author":{"email":"1e6780c0c00bdae1d869b533e91b45f21b7c79ee@gmail.com","name":"Neil Conway"},"message":"mesos: Update style guide for usage of std::atomic.\n\nmesos: Update style guide for usage of std::atomic.\n\nReview: https://reviews.apache.org/r/38265","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/941174914d309477b05865078ecf8a0f69db04f8"},{"sha":"4a01850c554048cfa53ca03d7d6e3cf901ce166d","author":{"email":"1e6780c0c00bdae1d869b533e91b45f21b7c79ee@gmail.com","name":"Neil Conway"},"message":"stout: Replace GCC intrinsics with std::atomic.\n\nMESOS-3326.\n\nReview: https://reviews.apache.org/r/37876","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/4a01850c554048cfa53ca03d7d6e3cf901ce166d"},{"sha":"4b938052b6af124eb1fdaec9b597c620627677ea","author":{"email":"1e6780c0c00bdae1d869b533e91b45f21b7c79ee@gmail.com","name":"Neil Conway"},"message":"libprocess: Replace GCC instrinsics and volatile with std::atomic.\n\nMESOS-3326.\n\nReview: https://reviews.apache.org/r/37877","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/4b938052b6af124eb1fdaec9b597c620627677ea"},{"sha":"87de003c6e8a4dfc2d29ae8b0aab1f83ff0c66a3","author":{"email":"1e6780c0c00bdae1d869b533e91b45f21b7c79ee@gmail.com","name":"Neil Conway"},"message":"mesos: Replace volatile with std::atomic.\n\nMESOS-3326.\n\nReview: https://reviews.apache.org/r/37878","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/87de003c6e8a4dfc2d29ae8b0aab1f83ff0c66a3"},{"sha":"3d082730b136d58f9c7dbdf615b66d904ed447c0","author":{"email":"0acc13d62cce94f85218ac2e2aa40486e36f9a9f@gmail.com","name":"Jie Yu"},"message":"Layers from Appc store should be the rootfs dir, not the top level dir.\n\nReview: https://reviews.apache.org/r/38284","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/3d082730b136d58f9c7dbdf615b66d904ed447c0"},{"sha":"05c608f80e3b120302bd2040780ce477677671dc","author":{"email":"e958ec5a56cd9794647c3623b3aa5a85122e74a5@jxu.me","name":"Jiang Yan Xu"},"message":"Fixed an issue which caused Appc provisioner not to clean up the the container dir for its provisioned rootfses.\n\n- Also changed the semantics of Provisioner::create() to fail with it fails to clean up unknown orphan containers.\n\nReview: https://reviews.apache.org/r/38319","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/05c608f80e3b120302bd2040780ce477677671dc"},{"sha":"bcf686b2bb1ede77602f0c6b3667c475c80a7967","author":{"email":"1e6780c0c00bdae1d869b533e91b45f21b7c79ee@gmail.com","name":"Neil Conway"},"message":"Added high-level description to persistent volume docs.\n\nReview: https://reviews.apache.org/r/38325","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/bcf686b2bb1ede77602f0c6b3667c475c80a7967"},{"sha":"c1b11acaf1af5133c707c8004bb6030a9b56e729","author":{"email":"8d7b3f579f19aea840973d9e194f8eeacca75bd7@gmail.com","name":"Joris Van Remoortere"},"message":"stout: Removed using-directives.\n\nReview: https://reviews.apache.org/r/38305","distinct":true,"url":"https://api.github.com/repos/jimenez/mesos/commits/c1b11acaf1af5133c707c8004bb6030a9b56e729"}]}