ISSUE-2439: Create BookieNettyServer after newBookie

[sijie]

Original Issue: apache#2439

---

When constructs BookieServer, newBookie maybe throw exception and then to shutdown nettyServer.

this.nettyServer = new BookieNettyServer(this.conf, null, allocator);
try {
      this.bookie = newBookie(conf, allocator, bookieServiceInfoProvider);
} catch (IOException | KeeperException | InterruptedException | BookieException e) {
     // interrupted on constructing a bookie
     this.nettyServer.shutdown();
     throw e;
}

Should we create BookieNettyServer after newBookie like below :

try {
      this.bookie = newBookie(conf, allocator, bookieServiceInfoProvider);
} catch (IOException | KeeperException | InterruptedException | BookieException e) {
      throw e;
}
final SecurityHandlerFactory shFactory = SecurityProviderFactoryFactory
      .getSecurityProviderFactory(conf.getTLSProviderFactoryClass());
this.requestProcessor = new BookieRequestProcessor(conf, bookie,
      statsLogger.scope(SERVER_SCOPE), shFactory, bookie.getAllocator());
this.nettyServer = new BookieNettyServer(this.conf, this.requestProcessor, allocator);