(DO NOT MERGE YET) Update logging to https://github.com/druid-io/druid-api/pull/27

Should go with https://github.com/druid-io/druid-api/pull/27
Crap, nice catch

On Sat, Dec 13, 2014 at 1:38 PM, xvrl <notifications@github.com> wrote:
>
> In
> indexing-service/src/main/java/io/druid/indexing/worker/http/WorkerResource.java
> <https://github.com/metamx/druid/pull/852#discussion-diff-21791502>:
>
> >
> >      if (stream.isPresent()) {
> > -      try {
> > -        return Response.ok(stream.get().getInput()).build();
> > +      try (InputStream logStream = stream.get().openStream()) {
>
> using a try-with-resources here broke task log streaming, because the
> stream gets closed when exiting the try-catch block instead of letting
> Jersey read out and close the inputstream
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/metamx/druid/pull/852/files#r21791502>.
>


-- 

<http://www.metamarkets.com/> Charles Allen PhDSr. Software Engineer|
METAMARKETS <http://www.metamarkets.com/>m 765.490.0454|t @drcrallen
<https://twitter.com/drcrallen>charles.allen@metamarkets.com