+1. Increased indentation is required not only for join, but for any operator which takes more than 1 inputs. Union / Union All would be another example.
On Mon, Apr 13, 2015 at 12:39 PM, Tomer Shiran <[email protected]> wrote: > +1 > > > On Fri, Apr 10, 2015 at 4:23 PM, Steven Phillips <[email protected]> > wrote: > > > I find the current format for physical plans, where we indent for every > > operator, but keep both sides of the join at the same indentation the > same, > > is very difficult to read, especially when we have several joins. > > > > I think it would be easier to read if we kept sequential operators at the > > same indentation, but increased indentation only when there is a join, > and > > include some marks to show the connection between the sides of the join > and > > the join operator. > > > > I created a gist with an example: > > > > https://gist.github.com/StevenMPhillips/74a6bf655175aabd14c0 > > > > > > > > -- > > Steven Phillips > > Software Engineer > > > > mapr.com > > >
