----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58512/ -----------------------------------------------------------
(Updated April 20, 2017, 5:40 p.m.) Review request for mesos, Anand Mazumdar, James Peach, and Jiang Yan Xu. Changes ------- Addressed review comments. Bugs: MESOS-7400 https://issues.apache.org/jira/browse/MESOS-7400 Repository: mesos Description ------- If the callback `on_headers_complete()` fails, the decoder's writer object is `None()`. So, when the callback `on_message_complete()` is called, we should not crash in that case. Note that the `CHECK(decoder->writer)` is valid for the callback `on_body()` is valid since this callback is called only on a success in `on_headers_complete()` callback. Diffs (updated) ----- 3rdparty/libprocess/src/decoder.hpp 4c779d42548958e610142438a57529ccb4478053 3rdparty/libprocess/src/tests/decoder_tests.cpp c0efef571815752cc28e5a0dc7a07adc82bb31d5 Diff: https://reviews.apache.org/r/58512/diff/4/ Changes: https://reviews.apache.org/r/58512/diff/3-4/ Testing ------- All tests passed. Thanks, Anindya Sinha