Cleanup a few scheduler warts.

Review Request #3032 - Created Oct. 24, 2015 and submitted

Information
John Sirois
pants
jsirois/issues/2413/cleanup
2413, 2453
6fbffd2...
Reviewers
pants-reviews
benjyw, ity, kwlzn, patricklaw, stuhood
Functions are now first class targets for a plan's execution.  The
example thrift planners are converted from tasks to functions to
exercise this.

Scheduler is now a proper `AbstractClass`.  A bad implementatiion of
`Plan.__getattr__` is fixed to allow this.  The same bad implementation
of `__getattr__` is also fixed in `Configuration`.

 src/python/pants/engine/exp/configuration.py         |   4 +-
 src/python/pants/engine/exp/examples/planners.py     |  48 ++++++++++++++----------
 src/python/pants/engine/exp/examples/visualizer.py   |  12 +++---
 src/python/pants/engine/exp/scheduler.py             | 101 +++++++++++++++++++++++++++++++++-----------------
 tests/python/pants_test/engine/exp/test_scheduler.py |  16 ++++----
 5 files changed, 112 insertions(+), 69 deletions(-)
CI went green here:
  https://travis-ci.org/pantsbuild/pants/builds/87148211

Files


John Sirois
John Sirois
John Sirois
John Sirois
Stu Hood
Stu Hood
John Sirois
John Sirois
John Sirois
John Sirois
Review request changed

Status: Closed (submitted)

Loading...