The Elasticsearch sink depends on Guava but this is not reflected in the POM.
SUCCESS: Integrated in flume-trunk #527 (See https://builds.apache.org/job/flume-trunk/527/)
FLUME-1679. Add dependency on Guava to flume-ng-elasticsearch-sink POM (hshreedharan: http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=58f3f6fb18e18fbf67fbd1ae0044c337845eba8d)
Committed, rev: 58f3f6fb18e18fbf67fbd1ae0044c337845eba8d. Thanks Andrew for the patch, and Ashish for rebasing it.
+1. Looks good. Committing.
Assigning to Andrew since he submitted the original patch
Rebased the patch and updated JIRA as Patch available. Trivial change, skipping review process
Got it! I had a different understanding, this makes sense. Thanks!
If the specific module requires it, it should be explicitly added (since the core module may, even though unlikely, get rid of it at some point, causing the build to fail). I will review this soon.
Hari ShreedharanMike Percy Can you please have a look at this JIRA. IMHO we can mark it as resolved. Please see my comments above.
Andrew Purtell] Guava dep is coming as a transitive dependency from flume-ng-core. As of now the build works fine without applying the patch. Can you please confirm if this is still an issue.
Head of 1.4 branch won't build without attached patch.