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

Review request for Flume.


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


Repository: flume-git


Description
-------

For now, flume-elasticsearch-sink is compiled with Elasticsearch-0.9.1, which 
was released in May 2013. During these two years, the Java API for 
Elasticsearch changes a lot, especially from version 0.9.x to 1.3.x. 
This lead to a problem for us: when we change the pom.xml and set the 
elasticsearch.version to a higher version, like 1.3.x, 1.4.x, 1.5.x, the old 
tests would fail or come out with errors. Some old APIs in the tests in 
elasticsearch sink are replaced by new API.
This patch fix all these compatibility problems in elasticsearch API in tests. 
With this new patch, we can compile flume-elasticsearch-sink with elasticsearch 
version 1.3.x, 1.4.x and1.5.x.
This patch also change the default elasticsearch.version to 1.5.2.


Diffs
-----

  
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/TestElasticSearchIndexRequestBuilderFactory.java
 8022111 
  
flume-ng-sinks/flume-ng-elasticsearch-sink/src/test/java/org/apache/flume/sink/elasticsearch/TestElasticSearchSink.java
 a58f344 
  pom.xml 64decfd 

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


Testing
-------


Thanks,

Weichen Ye

Reply via email to