----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38335/#review103231 -----------------------------------------------------------
Bad patch! Reviews applied: [38342, 38335] Failed command: ./support/apply-review.sh -n -r 38335 Error: 2015-10-20 08:29:53 URL:https://reviews.apache.org/r/38335/diff/raw/ [26873/26873] -> "38335.patch" [1] Successfully applied: Add JSON::protobuf for google::protobuf::RepeatedPtrField Currently, `stout/protobuf.hpp` provides a `JSON::Protobuf` utility which converts a `google::protobuf::Message` into a `JSON::Object`. We should add the support for `google::protobuf::RepeatedPtrField<T>` by introducing overloaded functions. Review: https://reviews.apache.org/r/38335 Checking 24 files using filter --filter=-,+build/class,+build/deprecated,+build/endif_comment,+readability/todo,+readability/namespace,+runtime/vlog,+whitespace/blank_line,+whitespace/comma,+whitespace/end_of_line,+whitespace/ending_newline,+whitespace/forcolon,+whitespace/indent,+whitespace/line_length,+whitespace/operators,+whitespace/semicolon,+whitespace/tab,+whitespace/todo src/tests/reservation_endpoints_tests.cpp:830: Lines should be <= 80 characters long [whitespace/line_length] [2] Total errors found: 1 Failed to commit patch - Mesos ReviewBot On Oct. 20, 2015, 5:37 a.m., Klaus Ma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38335/ > ----------------------------------------------------------- > > (Updated Oct. 20, 2015, 5:37 a.m.) > > > Review request for mesos, Alexander Rukletsov, Michael Park, and Jan Schlicht. > > > Bugs: MESOS-3405 > https://issues.apache.org/jira/browse/MESOS-3405 > > > Repository: mesos > > > Description > ------- > > Currently, `stout/protobuf.hpp` provides a `JSON::Protobuf` utility which > converts a `google::protobuf::Message` into a `JSON::Object`. > We should add the support for `google::protobuf::RepeatedPtrField<T>` by > introducing overloaded functions. > > > Diffs > ----- > > src/common/http.cpp f56d8a1 > src/docker/executor.cpp 1e49013 > src/examples/persistent_volume_framework.cpp 176ac3d > src/launcher/executor.cpp 50b3c6e > src/master/contender.cpp c641305 > src/master/http.cpp 093f793 > src/master/maintenance.cpp 5fe9358 > src/master/registrar.cpp 1117232 > src/slave/containerizer/fetcher.cpp e0d02d5 > src/slave/containerizer/mesos/containerizer.cpp d1fc5a4 > src/slave/monitor.cpp aa6e958 > src/tests/containerizer/launch_tests.cpp de655ec > src/tests/fault_tolerance_tests.cpp f78a291 > src/tests/master_maintenance_tests.cpp e89ce3b > src/tests/master_tests.cpp ee24739 > src/tests/mesos.hpp 3e58b45 > src/tests/mesos.cpp ab2d85b > src/tests/monitor_tests.cpp 583e711 > src/tests/reservation_endpoints_tests.cpp f5f9c48 > src/tests/resources_tests.cpp 6584fc6 > src/tests/scheduler_http_api_tests.cpp d338a1b > src/tests/script.cpp bcc1fab > src/tests/slave_tests.cpp 10a4fa7 > src/usage/main.cpp 86fd796 > > Diff: https://reviews.apache.org/r/38335/diff/ > > > Testing > ------- > > make > make check > > > Thanks, > > Klaus Ma > >