-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49458/
-----------------------------------------------------------

(Updated July 1, 2016, 2:40 p.m.)


Review request for Flume.


Bugs: FLUME-2921
    https://issues.apache.org/jira/browse/FLUME-2921


Repository: flume-git


Description
-------

This patch adds the support for Elasticsearch version 2.0+. The version I used 
is 2.3.3, which is the latest stable release. This also required upgrading 
guava.
This patch does not fix any known issues with this sink, its only purpose is to 
support current versions of elasticsearch.


Diffs (updated)
-----

  flume-ng-channels/flume-spillable-memory-channel/pom.xml 60e49d2 
  flume-ng-core/pom.xml 41586dd 
  flume-ng-sinks/flume-ng-elasticsearch-sink/pom.xml c372c0b 
  
flume-ng-sinks/flume-ng-elasticsearch-sink/src/main/java/org/apache/flume/sink/elasticsearch/ContentBuilderUtil.java
 83c3ffd 
  
flume-ng-sinks/flume-ng-elasticsearch-sink/src/main/java/org/apache/flume/sink/elasticsearch/ElasticSearchLogStashEventSerializer.java
 3638368 
  
flume-ng-sinks/flume-ng-elasticsearch-sink/src/main/java/org/apache/flume/sink/elasticsearch/client/ElasticSearchTransportClient.java
 2cf365e 
  
flume-ng-sinks/flume-ng-elasticsearch-sink/src/test/java/org/apache/flume/sink/elasticsearch/AbstractElasticSearchSinkTest.java
 f9272fa 
  
flume-ng-sinks/flume-ng-elasticsearch-sink/src/test/java/org/apache/flume/sink/elasticsearch/TestElasticSearchDynamicSerializer.java
 d4e4654 
  
flume-ng-sinks/flume-ng-elasticsearch-sink/src/test/java/org/apache/flume/sink/elasticsearch/TestElasticSearchIndexRequestBuilderFactory.java
 8022111 
  
flume-ng-sinks/flume-ng-elasticsearch-sink/src/test/java/org/apache/flume/sink/elasticsearch/TestElasticSearchLogStashEventSerializer.java
 ab9587d 
  
flume-ng-sinks/flume-ng-elasticsearch-sink/src/test/java/org/apache/flume/sink/elasticsearch/TestElasticSearchSink.java
 a58f344 
  pom.xml 85c0dc8 

Diff: https://reviews.apache.org/r/49458/diff/


Testing
-------

I made sure that all unit tests (due to guava upgrade) pass successfully. The 
known flaky tests may not pass, though.


Thanks,

Lior Zeno

Reply via email to