ianoc
commented
7 years ago all seems pretty lgtm, like all the sealed traits. Code looks nice and readable too 👍
(Overall these plots actually I think are making the planning code overall more readable so thats pretty awesome too)
This is a follow on to #1666 to bring almost all of the cascading planning out of Grouped related to reduce steps. There is still a small bit which overlaps with CoGrouped, which will be moved after we tackle CoGrouped.
r? @fwbrasil @piyushnarang @ianoc