[TACHYON-62] Netty

Netty implementation of the data server.

This patch is not done yet.  Its intent is to show the work I have right now for commenting.

This PR should not be merged until performance results come in.
Yep.  That's the current style in tachyon.
On Aug 19, 2014 11:59 PM, "Reynold Xin" <notifications@github.com> wrote:

> In core/src/main/java/tachyon/worker/netty/DataServerHandler.java:
>
> > +
> > +import io.netty.channel.ChannelFuture;
> > +import io.netty.channel.ChannelFutureListener;
> > +import io.netty.channel.ChannelHandler;
> > +import io.netty.channel.ChannelHandlerContext;
> > +import io.netty.channel.ChannelInboundHandlerAdapter;
> > +
> > +/**
> > + * Main logic for the read path. This class consumes {@link tachyon.worker.netty.BlockRequest}
> > + * messages and returns {@link tachyon.worker.netty.BlockResponse} messages.
> > + */
> > +@ChannelHandler.Sharable
> > +public final class DataServerHandler extends ChannelInboundHandlerAdapter {
> > +  private static final Logger LOG = Logger.getLogger(DataServerHandler.class);
> > +
> > +  private final BlocksLocker LOCKER;
>
> is the convention in tachyon to use upper case everything for final
> variables? seems strange to me
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/amplab/tachyon/pull/333/files#r16461206>.
>