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 2020/06/02 00:10:05 UTC

[GitHub] [druid] surekhasaharan commented on pull request #9717: remove DruidLeaderClient.goAsync(..) that does not follow redirect. Replace its usage by DruidLeaderClient.go(..) with InputStreamFullResponseHandler

surekhasaharan commented on pull request #9717:
URL: https://github.com/apache/druid/pull/9717#issuecomment-637191104


   LGTM, thanks @himanshug 
   Though it seems CI is failing because of code coverage
   ```
   ERROR: Insufficient line coverage of 20% (10/50). Required 65%.
   ERROR: Insufficient branch coverage of 33% (2/6). Required 65%.
   ```


----------------------------------------------------------------
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



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