twitter / scalding

A Scala API for Cascading
http://twitter.com/scalding
Apache License 2.0
3.5k stars 706 forks source link

Fix issue 1786 #1788

Closed johnynek closed 6 years ago

johnynek commented 6 years ago

Cascading has some requirements of the input graphs. So, toPipeUnoptimized still needs to apply some correctness rules (mostly regarding merges which are seemingly somewhat touchy, and historically hashJoin has also had issues that maybe should migrate to that point).

closes #1786

non commented 6 years ago

Seems reasonable to me. 👍