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.
Thanks for the feedback.

I was planning to make a few of the changes proposed after testing the
current performance vs master.

The idea is that master is mostly the same as this patch so gives a one to
one comparison.

Netty Best Practices: http://youtu.be/WsMOJqAYW5M

After I get the base line, will try each optimization one by one to see the
change.
On Aug 11, 2014 11:46 PM, "Mubarak Seyed" <notifications@github.com> wrote:

> In core/src/main/java/tachyon/worker/netty/NettyDataServer.java:
>
> > +        new ServerBootstrap()
> > +            .group(BOSS_GROUP, WORKER_GROUP)
> > +            //TODO support EpollSocketChannel for max performance on Linux
> > +            .channel(NioServerSocketChannel.class)
> > +            .handler(new LoggingHandler(LogLevel.DEBUG))
> > +            .childHandler(childHandler)
> > +            .option(ChannelOption.SO_BACKLOG, 1024)
> > +            .childOption(ChannelOption.SO_KEEPALIVE, true);
> > +
> > +    this.CHANNEL_FUTURE = bootstrap.bind(address).sync();
> > +  }
> > +
> > +  @Override
> > +  public void close() throws IOException {
> > +    WORKER_GROUP.shutdownGracefully();
> > +    BOSS_GROUP.shutdownGracefully();
>
> I think BOSS_GROUP shutdown first?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/amplab/tachyon/pull/333/files#r16097832>.
>