----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67185/#review207011 -----------------------------------------------------------
src/python/lib/mesos/http.py Lines 41 (patched) <https://reviews.apache.org/r/67185/#comment290175> not that you've mentioned it, we actually heavily use https://github.com/timothycrosley/isort at Uber. would you guys be open to adopting it as well? src/python/lib/mesos/http.py Lines 196 (patched) <https://reviews.apache.org/r/67185/#comment290176> the name i guess is a little misleading. what this does is to encode a message as either protobuf or json, determined by `self.default_encoding`. can you think of a better name for it? src/python/lib/mesos/http.py Lines 345 (patched) <https://reviews.apache.org/r/67185/#comment290177> indeed it should - Eric Chung On June 28, 2018, 6:17 a.m., Eric Chung wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67185/ > ----------------------------------------------------------- > > (Updated June 28, 2018, 6:17 a.m.) > > > Review request for mesos, Armand Grillet, Jason Lai, Kevin Klues, and Zhitao > Li. > > > Repository: mesos > > > Description > ------- > > This change adds the method `Resource.request_protobuf()` which assumes > that both the request and response data are encoded in protobuf. This > will be used later in the RPC client, which will also include a decoding > step. > > Testing Done: > Ran unit tests with `tox` under `src/python/lib` > > > Diffs > ----- > > src/python/lib/mesos/constants.py PRE-CREATION > src/python/lib/mesos/http.py 073c159dc6916fa5d7349a033db022d7175aef05 > src/python/lib/requirements.in 0742f3d846c99c1c4907d9628fb49845564563b2 > src/python/lib/tests/test_http.py 66dd6d7c6272d1828dd591829ca3543d3430f69b > > > Diff: https://reviews.apache.org/r/67185/diff/4/ > > > Testing > ------- > > > Thanks, > > Eric Chung > >