----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9306/#review16371 -----------------------------------------------------------
Ship it! src/linux/cgroups.cpp <https://reviews.apache.org/r/9306/#comment34838> Can't the implicit constructor work in here? src/linux/cgroups.cpp <https://reviews.apache.org/r/9306/#comment34839> ditto src/linux/cgroups.cpp <https://reviews.apache.org/r/9306/#comment34840> ditto src/linux/cgroups.cpp <https://reviews.apache.org/r/9306/#comment34841> ditto third_party/libprocess/include/stout/gzip.hpp <https://reviews.apache.org/r/9306/#comment34842> thanks - Ben Mahler On Feb. 8, 2013, 5:49 a.m., Benjamin Hindman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9306/ > ----------------------------------------------------------- > > (Updated Feb. 8, 2013, 5:49 a.m.) > > > Review request for mesos, Vinod Kone and Ben Mahler. > > > Description > ------- > > See summary. > > > Diffs > ----- > > src/common/values.cpp 8aebc0b57de110e14c55c5b6d543f17165cfd397 > src/detector/detector.cpp ca921151a1fc1a57d265b3439d07a8c45e920f21 > src/files/files.cpp 60b567eb62e84ccc99b0b1978733a0ea96560813 > src/flags/parse.hpp df7e3480c1f5422d185c86cc5f34e7b87b024a2f > src/linux/cgroups.cpp 03b31e7309b9dd65f00d3b0da2abb81ddaaeea43 > src/linux/fs.cpp ea74c3b935dc716fd53827c8a7eb7010961e7aa6 > src/linux/proc.cpp 8a0fc48dc9769df35d682ece477246b2df2fc0d4 > src/log/coordinator.cpp d9f67bd8b07e6f8b3e022c21f1764d298e314594 > src/log/log.hpp 151eebd5d665993dfaac399553a9a315c093762b > src/log/replica.cpp 392cb15991657d8e79637e1c4bc21ec813e3117d > src/logging/logging.cpp e9d778f840f2d6713dbb99d0a8844033be9f0e5f > src/master/frameworks_manager.cpp 189a7e3b4811643baf285ad29f939453257c705f > src/sched/sched.cpp f1eeab6f12ee300d77013c6a4ba62ccd7fdb0d1d > src/slave/cgroups_isolation_module.cpp > 63cefc33cf34eebb82db5d8448b751be8652fa36 > src/state/leveldb.cpp 6137b5d089dc9ea4b60fd56153b72e14691df3d8 > src/state/serializer.hpp 1915cefb102526fbad330b090b8c788bea65ed84 > src/state/zookeeper.cpp 4c684797791bb2158d31573d2d47b0da70283bc6 > src/tests/master_tests.cpp 948ab5dff34eeba1f3ce593a864ddf282c8b19ed > src/tests/stout_tests.cpp fffef3e42774e9b090a098a219a92e642d303d27 > src/zookeeper/group.cpp e74538e3dfa6951178f3e4b49dfd94374a0aa28c > src/zookeeper/url.hpp c62539b30503a94ed82e11fe84d988c122d36bb6 > third_party/libprocess/include/process/http.hpp > 40248dab3df59b322ad9c5a4858961ef471779fd > third_party/libprocess/include/stout/duration.hpp > caf7ce42b34dfb74206d1f10aafabaaec47e7dfc > third_party/libprocess/include/stout/error.hpp PRE-CREATION > third_party/libprocess/include/stout/format.hpp > fc349903d31d6e31a8147e3a6be87b37698f28f0 > third_party/libprocess/include/stout/fs.hpp > 9e62a1b91bc9fac092818ffb3c8bcec46b0bd26d > third_party/libprocess/include/stout/gzip.hpp > 6ff1288683a0d3b05098c772945cc2a8b73f8b01 > third_party/libprocess/include/stout/net.hpp > 1909046017da005a5e7c64d525bb3c741d5ad436 > third_party/libprocess/include/stout/numify.hpp > 2a59c9269902541d82ba86a2d159984f5bbc6b73 > third_party/libprocess/include/stout/os.hpp > 712fb209061384b8c045875ef8898a6efd778514 > third_party/libprocess/include/stout/protobuf.hpp > 25d4634bbc51d14ea78c1a922353f0ce52e458df > > Diff: https://reviews.apache.org/r/9306/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Benjamin Hindman > >
