STREAMS-456: PR feedback

Project: http://git-wip-us.apache.org/repos/asf/incubator-streams/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-streams/commit/f597411a
Tree: http://git-wip-us.apache.org/repos/asf/incubator-streams/tree/f597411a
Diff: http://git-wip-us.apache.org/repos/asf/incubator-streams/diff/f597411a

Branch: refs/heads/master
Commit: f597411a01458a32b3c4e9fad8311de3180b4b41
Parents: 1799fbf
Author: Steve Blackmon @steveblackmon <sblack...@apache.org>
Authored: Wed Nov 16 02:04:04 2016 +0100
Committer: Steve Blackmon @steveblackmon <sblack...@apache.org>
Committed: Wed Nov 16 02:04:04 2016 +0100

----------------------------------------------------------------------
 .../facebook/provider/page/FacebookPageDataCollector.java        | 4 ++--
 .../streams/facebook/provider/page/FacebookPageProvider.java     | 1 -
 2 files changed, 2 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-streams/blob/f597411a/streams-contrib/streams-provider-facebook/src/main/java/org/apache/streams/facebook/provider/page/FacebookPageDataCollector.java
----------------------------------------------------------------------
diff --git 
a/streams-contrib/streams-provider-facebook/src/main/java/org/apache/streams/facebook/provider/page/FacebookPageDataCollector.java
 
b/streams-contrib/streams-provider-facebook/src/main/java/org/apache/streams/facebook/provider/page/FacebookPageDataCollector.java
index ca68a7e..0e88dd4 100644
--- 
a/streams-contrib/streams-provider-facebook/src/main/java/org/apache/streams/facebook/provider/page/FacebookPageDataCollector.java
+++ 
b/streams-contrib/streams-provider-facebook/src/main/java/org/apache/streams/facebook/provider/page/FacebookPageDataCollector.java
@@ -27,8 +27,8 @@ import org.apache.streams.jackson.StreamsJacksonMapper;
 
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.google.common.annotations.VisibleForTesting;
-import com.google.common.base.Joiner;
 
+import org.apache.commons.lang.StringUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -52,7 +52,7 @@ public class FacebookPageDataCollector extends 
FacebookDataCollector {
 
   public FacebookPageDataCollector(BlockingQueue<StreamsDatum> queue, 
FacebookPageProviderConfiguration configuration) {
     super(configuration, queue);
-    fields = Joiner.on(',').join(configuration.getFields());
+    fields = StringUtils.join(configuration.getFields(), ',');
   }
 
   @Override

http://git-wip-us.apache.org/repos/asf/incubator-streams/blob/f597411a/streams-contrib/streams-provider-facebook/src/main/java/org/apache/streams/facebook/provider/page/FacebookPageProvider.java
----------------------------------------------------------------------
diff --git 
a/streams-contrib/streams-provider-facebook/src/main/java/org/apache/streams/facebook/provider/page/FacebookPageProvider.java
 
b/streams-contrib/streams-provider-facebook/src/main/java/org/apache/streams/facebook/provider/page/FacebookPageProvider.java
index cd74927..d11a486 100644
--- 
a/streams-contrib/streams-provider-facebook/src/main/java/org/apache/streams/facebook/provider/page/FacebookPageProvider.java
+++ 
b/streams-contrib/streams-provider-facebook/src/main/java/org/apache/streams/facebook/provider/page/FacebookPageProvider.java
@@ -31,7 +31,6 @@ import org.apache.streams.jackson.StreamsJacksonMapper;
 import com.fasterxml.jackson.core.JsonProcessingException;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.google.common.annotations.VisibleForTesting;
-import com.google.common.base.Joiner;
 import com.google.common.base.Preconditions;
 import com.google.common.util.concurrent.Uninterruptibles;
 import com.typesafe.config.Config;

Reply via email to