Except the blanks lines I agree with David. It is future proof to have everything in lexicographic order.
Will still like to have blank lines. Chandni On Wed, Apr 6, 2016 at 12:24 PM, David Yan <[email protected]> wrote: > I think the import order should be alphabetical or lexicographical, and no > blank lines are needed. > > The reason is this: > > 1) It's simple and future proof, and there will be no need to have this > discussion again in the future about what order it should be and where the > blank lines should be placed > 2) It's easy to follow, and there is no rule to remember other than it's > alphabetical > 3) It's easy to find where a particular import lines are because it's > alphabetical > > Whatever we come up with, we should make the change ASAP and fix all the > style violations once and for all. > > David > > > On Tue, Apr 5, 2016 at 5:12 PM, Chandni Singh <[email protected]> > wrote: > > > Hi, > > > > We need to include org.apache.apex in the import order that we have right > > now: > > > > *Current order:* > > java > > javax > > org > > org.apache > > io > > com > > com.datatorrent > > * > > all static imports > > > > *Proposed order:* > > java > > javax > > org > > org.apache > > io > > com > > com.datatorrent > > org.apache.apex > > * > > all static imports > > > > Thanks, > > Chandni > > >
