The Elasticsearch sink depends on Guava but this is not reflected in the POM.
Head of 1.4 branch won't build without attached patch.
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.
Hari ShreedharanMike Percy Can you please have a look at this JIRA. IMHO we can mark it as resolved. Please see my comments above.
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.
Got it! I had a different understanding, this makes sense. Thanks!
Rebased the patch and updated JIRA as Patch available. Trivial change, skipping review process
Assigning to Andrew since he submitted the original patch
+1. Looks good. Committing.
Committed, rev: 58f3f6fb18e18fbf67fbd1ae0044c337845eba8d. Thanks Andrew for the patch, and Ashish for rebasing it.
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)