[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.
Not clear what zero would mean for Netty.  Without seeing it should get the
default
On Aug 28, 2014 12:28 AM, "Mubarak Seyed" <notifications@github.com> wrote:

> In core/src/main/java/tachyon/conf/WorkerConf.java:
>
> > @@ -77,5 +89,20 @@ private WorkerConf() {
> >      WORKER_CHECKPOINT_THREADS = getIntProperty("tachyon.worker.checkpoint.threads", 1);
> >      WORKER_PER_THREAD_CHECKPOINT_CAP_MB_SEC =
> >          getIntProperty("tachyon.worker.per.thread.checkpoint.cap.mb.sec", Constants.SECOND_MS);
> > +
> > +    NETWORK_TYPE = getEnumProperty("tachyon.worker.network.type", NetworkType.NETTY);
> > +    NETTY_CHANNEL_TYPE =
> > +        getEnumProperty("tachyon.worker.network.netty.channel", ChannelType.defaultType());
> > +
> > +    NETTY_DATA_PROCESS_THREADS =
> > +        getIntProperty("tachyon.worker.network.netty.process.threads", Runtime.getRuntime()
> > +            .availableProcessors());
> > +
> > +    NETTY_HIGH_WATER_MARK =
> > +        getIntProperty("tachyon.worker.network.netty.watermark.high", 32 * 1024);
> > +    NETTY_LOW_WATER_MARK = getIntProperty("tachyon.worker.network.netty.watermark.low", 8 * 1024);
> > +    NETTY_BACKLOG = getIntegerProperty("tachyon.worker.network.netty.backlog", null);
>
> I think default backlog value is OS specific. Can't we just set it to 0 as
> default value so that default implementation value will be used.
>
>
> http://download.java.net/jdk7/archive/b123/docs/api/java/net/ServerSocket.html#ServerSocket(int,
> int)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/amplab/tachyon/pull/333/files#r16824613>.
>