reuse cluster for TFsShellTest and made each test isolated

Cut TFsShellTest from 6 minutes to 30 seconds.  Same trick can be used for all long running tests.
" then as *alphabetical **order*"

When was this added? We talked about the need to justify changes to Google
style. In Google style the logic is of ordering is about flow. Alphabet
order does not improve readability.

Why was this change added and where can I find the justifications for it?
On Dec 21, 2014 11:19 AM, "Haoyuan Li" <notifications@github.com> wrote:

> In core/src/test/java/tachyon/client/FileInStreamTest.java
> <https://github.com/amplab/tachyon/pull/570#discussion-diff-22149561>:
>
> >      System.setProperty("tachyon.user.quota.unit.bytes", "1000");
> >      System.setProperty("tachyon.user.default.block.size.byte", String.valueOf(BLOCK_SIZE));
> > -    mLocalTachyonCluster = new LocalTachyonCluster(10000);
> > -    mLocalTachyonCluster.start();
> > -    mTfs = mLocalTachyonCluster.getClient();
> > +    CLUSTER = new LocalTachyonCluster(10000);
> > +    CLUSTER.start();
> > +    TFS = CLUSTER.getClient();
> > +  }
> > +
>
> Class and member modifiers, when present, appear in the order recommended
> by the Java Language Specification: public protected private abstract
> static final transient volatile synchronized native strictfp, then as
> alphabetical order.
>
> http://tachyon-project.org/master/Startup-Tasks-for-New-Contributors.html
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/amplab/tachyon/pull/570/files#r22149561>.
>