mesosphere / marathon-lb

Marathon-lb is a service discovery & load balancing tool for DC/OS
Apache License 2.0
449 stars 300 forks source link

Decouple zdd.py into Python modules #292

Open mieciu opened 8 years ago

mieciu commented 8 years ago

Hi,

I've been recently digging through zdd.py code to make sure I understand everything correct and came into idea of refactoring this over 800L Python script.

I've done some initial work, but I'd like to hear a green light from project maintainers and developers that this is something that matches projects direction.

An idea is to simply decouple long script into set of python modules, more or less just as in screenshot attached: zdd_py_-_marathon-lb_-____projects_marathon-lb_

I'd love to hear your opinion!

brndnmtthws commented 8 years ago

Sounds reasonable to me. Please keep any refactoring code changes separate from functionality changes (in separate PRs).