[ 
https://issues.apache.org/jira/browse/FLUME-2957?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16353743#comment-16353743
 ] 

Hudson commented on FLUME-2957:
-------------------------------

SUCCESS: Integrated in Jenkins build Flume-trunk-hbase-1 #340 (See 
[https://builds.apache.org/job/Flume-trunk-hbase-1/340/])
FLUME-2957. Remove Guava from our public API (szaboferee: 
[http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=7f85df9e473ee675d461d5b76650694c5a6c0088])
* (edit) 
flume-ng-core/src/main/java/org/apache/flume/source/MultiportSyslogTCPSource.java
* (edit) 
flume-ng-node/src/main/java/org/apache/flume/node/MaterializedConfiguration.java
* (edit) 
flume-ng-sinks/flume-http-sink/src/main/java/org/apache/flume/sink/http/HttpSink.java
* (edit) flume-ng-configuration/src/main/java/org/apache/flume/Context.java
* (edit) 
flume-ng-node/src/main/java/org/apache/flume/node/SimpleMaterializedConfiguration.java
* (edit) 
flume-ng-embedded-agent/src/test/java/org/apache/flume/agent/embedded/TestEmbeddedAgentEmbeddedSource.java


> Remove Guava from our public API
> --------------------------------
>
>                 Key: FLUME-2957
>                 URL: https://issues.apache.org/jira/browse/FLUME-2957
>             Project: Flume
>          Issue Type: Task
>    Affects Versions: 1.8.0
>            Reporter: Lior Zeno
>            Assignee: Denes Arvay
>            Priority: Major
>              Labels: breaking_change
>             Fix For: 2.0.0
>
>
> Context.getParameters (flume-ng-configuration module) returns 
> com.google.common.collect.ImmutableMap (Guava). We should clean our API and 
> return either a native java interface or Flume's.
> In addition to the current state being a bad practice, this also means that 
> we are unable to shade Guava in Flume.
> Note: Since this breaks our public API, I'll reschedule this issue to 2.0 
> once we have this version managed in jira.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to