Github user cloud-fan commented on a diff in the pull request:
    --- Diff: 
    @@ -274,3 +279,7 @@ abstract class BinaryNode extends LogicalPlan {
       override final def children: Seq[LogicalPlan] = Seq(left, right)
    +abstract class KeepOrderUnaryNode extends UnaryNode {
    --- End diff --
    OrderPreservingUnaryNode sounds better.
    It only makes sense for unary node, so I don't think mixin trait is a good 


To unsubscribe, e-mail:
For additional commands, e-mail:

Reply via email to