Good catch. That was meant to be removed after debugging a Kafka log issue

Sent from my iPhone

On Apr 30, 2013, at 7:26 PM, Nitin Motgi <notifications@github.com> wrote:

> In src/main/java/com/continuuity/weave/internal/yarn/YarnWeavePreparer.java:
> 
> > +                                                            .add("org.xerial.snappy")
> > +                                                            .build()
> > +                                                          );
> > +      resourceCleaner.add(createAppMasterJar(bundler, localResources));
> > +      resourceCleaner.add(createContainerJar(bundler, localResources));
> > +      resourceCleaner.add(createLogBackTemplate(localResources));
> > +      resourceCleaner.add(saveWeaveSpec(weaveSpec, localResources));
> > +      resourceCleaner.add(populateRunnableResources(weaveSpec, localResources));
> > +
> > +      ContainerLaunchContext containerLaunchContext = Records.newRecord(ContainerLaunchContext.class);
> > +      containerLaunchContext.setLocalResources(localResources);
> > +
> > +      RunId runId = RunIds.generate();
> > +      containerLaunchContext.setCommands(ImmutableList.of(
> > +        "java", "-cp", "appMaster.jar", ApplicationMasterMain.class.getName(),
> > +        "1>/tmp/out", "2>/tmp/err"
> It's better not to have it to forward to tmp. If someone does system.out it would get there and might fill up temp. If you want to ignore should be move it to /dev/null ?
> 
> —
> Reply to this email directly or view it on GitHub.