From 336db3277b65bfd980f2f5122239b34ee62922ec Mon Sep 17 00:00:00 2001 From: Rick Jensen Date: Thu, 30 Mar 2017 13:09:43 -0500 Subject: [PATCH] Make druid client follow redirects by default --- CHANGELOG.md | 3 +++ .../webservice/druid/client/impl/AsyncDruidWebServiceImpl.java | 1 + 2 files changed, 4 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 243aee3289..c7d6e998b4 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -133,6 +133,9 @@ Current ### Fixed: +- [Default the AsyncDruidWebServiceImpl to follow redirects](https://github.com/yahoo/fili/pull/214) + * It defaulted to not following redirects, and now it doesn't + - [Reenable custom query types in TestDruidWebService]() - [Fixed Segment Metadata Loader Unconfigured Dimension Bug](https://github.com/yahoo/fili/pull/197) diff --git a/fili-core/src/main/java/com/yahoo/bard/webservice/druid/client/impl/AsyncDruidWebServiceImpl.java b/fili-core/src/main/java/com/yahoo/bard/webservice/druid/client/impl/AsyncDruidWebServiceImpl.java index 1761a78549..ef68fb55c1 100644 --- a/fili-core/src/main/java/com/yahoo/bard/webservice/druid/client/impl/AsyncDruidWebServiceImpl.java +++ b/fili-core/src/main/java/com/yahoo/bard/webservice/druid/client/impl/AsyncDruidWebServiceImpl.java @@ -115,6 +115,7 @@ private static AsyncHttpClient initializeWebClient(int requestTimeout) { .setRequestTimeout(requestTimeout) .setConnectionTtl(requestTimeout) .setPooledConnectionIdleTimeout(requestTimeout) + .setFollowRedirect(true) .build(); return new DefaultAsyncHttpClient(config);