douban / pymesos

A pure python implementation of Mesos scheduler and executor
BSD 3-Clause "New" or "Revised" License
163 stars 88 forks source link

Fix new-master msg handler (MESOS-3135) #15

Closed vshlapakov closed 8 years ago

vshlapakov commented 8 years ago

The current pymesos version (#a6a64bac50bf) doesn't work well for Mesos >=0.24 due to https://issues.apache.org/jira/browse/MESOS-3135 (ZK Mesos master data serialization with JSON instead of protocol buff).

The PR implements a necessary fix to parse ZK data properly for last Mesos versions.

Review, please, and let me know what do you think about it.

Review on Reviewable

mckelvin commented 8 years ago

@windreamer Please review.

windreamer commented 8 years ago

LGTM, thank you @vshlapakov .