You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2019/09/23 20:40:08 UTC

[GitHub] [incubator-druid] clintropolis opened a new pull request #8576: fix http firehose factory leaky connection in constructor

clintropolis opened a new pull request #8576: fix http firehose factory leaky connection in constructor
URL: https://github.com/apache/incubator-druid/pull/8576
 
 
   ### Description
   This PR removes a leaky and unnecessary URL connection from the HTTP firehose factory constructor, pushing it into the `openObjectStream` method which is the only place it was used.
   
   The bug was originally introduced in https://github.com/apache/incubator-druid/pull/5162
   
   <hr>
   
   This PR has:
   - [x] been self-reviewed.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org