introduce an import style convention

[akiezun]

without an import style convention, many commits include edits that change full imports to star (*) back and forth.

I vote for full name imports everywhere, it's easy to collapse them in the editor.

[mattsooknah]

I agree w/ Adam. Full names are more specific, avoid namespace collisions, and (depending on package sizes) could reduce compile time a bit (maybe insignificant for us though).

[lbergelson]

I don't feel strongly either way. I'm fine with full imports.

[lbergelson]

By default it looks like intellij switches to import a package with * when 5 or more classes from the same package are imported. You can change it to an arbitrary number in preference->editor->codes style-> java->imports

Do we want to set it higher? The only issue I see is that 50% of our code is going to be import java.utils lines. You can also specify specific packages to import with *, we could import all on java standard library and import specifically when using external libraries.

[droazen]

I vote for collapsing after 5 or more -- full imports would clutter the diffs during code review. Also, gains in compile time are likely to not be noticeable.

[droazen]

Brain not fully working when I wrote the above -- of course both options clutter the diffs in different ways :)

[droazen]

This is done -- we've standardized on collapsing after 5 or more.