Filed YARN-760. Thanks, Vinod.
On Mon, Jun 3, 2013 at 9:40 PM, Sandy Ryza <[email protected]> wrote: > Ah ok, makes sense. I'll file a JIRA. > > -Sandy > > > On Mon, Jun 3, 2013 at 9:37 PM, Vinod Kumar Vavilapalli < > [email protected]> wrote: > >> >> I saw that one recently too, but forgot to file a JIRA. >> >> Long time back (before we first pushed YARN code out into the open?), we >> had avro as our serialization and had it all over the place. Then we >> abstracted it out and then eventually moved to PB. >> >> Should be fixed. >> >> Thanks, >> +Vinod >> >> On Jun 3, 2013, at 9:33 PM, Sandy Ryza wrote: >> >> > Does anybody know why the NodeManager wraps whatever exception its >> throwing >> > in an AvroRuntimeException? >> > >> > --- >> > @Override >> > public void start() { >> > >> > // NodeManager is the last service to start, so NodeId is available. >> > this.nodeId = this.context.getNodeId(); >> > >> > InetSocketAddress httpBindAddress = getConfig().getSocketAddr( >> > YarnConfiguration.NM_WEBAPP_ADDRESS, >> > YarnConfiguration.DEFAULT_NM_WEBAPP_ADDRESS, >> > YarnConfiguration.DEFAULT_NM_WEBAPP_PORT); >> > try { >> > this.httpPort = httpBindAddress.getPort(); >> > // Registration has to be in start so that ContainerManager can get >> > the >> > // perNM tokens needed to authenticate ContainerTokens. >> > registerWithRM(); >> > super.start(); >> > startStatusUpdater(); >> > } catch (Exception e) { >> > throw new AvroRuntimeException(e); >> > } >> > } >> > --- >> > >> > any guidance appreciated, >> > -Sandy >> >> >
