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



Thanks for your contribution!

There is an open ticket for bumping java version to 8. Will your patch change 
much if we upgrade to the latest Jetty version?

- Lior Zeno


On Oct. 17, 2016, 7:29 p.m., Tristan Stevens wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52955/
> -----------------------------------------------------------
> 
> (Updated Oct. 17, 2016, 7:29 p.m.)
> 
> 
> Review request for Flume.
> 
> 
> Repository: flume-git
> 
> 
> Description
> -------
> 
> This patch upgrades the HTTPSource and HTTP Metrics server to Jetty 9.2 (N.B. 
> 9.3/9.4 depend on JDK8). It also adds the ability to further configure the 
> Jetty server and also run in multi-threaded mode.
> 
> 
> Diffs
> -----
> 
>   flume-ng-core/pom.xml 2262d94 
>   
> flume-ng-core/src/main/java/org/apache/flume/instrumentation/MonitoringType.java
>  4e1a28c 
>   
> flume-ng-core/src/main/java/org/apache/flume/instrumentation/http/HTTPMetricsServer.java
>  921a1f7 
>   flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSource.java 
> 38bdfda 
>   
> flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSourceConfigurationConstants.java
>  86caf7d 
>   
> flume-ng-core/src/main/java/org/apache/flume/tools/FlumeBeanConfigurator.java 
> PRE-CREATION 
>   
> flume-ng-core/src/main/java/org/apache/flume/tools/HTTPServerConstraintUtil.java
>  479cfc4 
>   
> flume-ng-core/src/test/java/org/apache/flume/instrumentation/http/TestHTTPMetricsServer.java
>  09d419f 
>   
> flume-ng-core/src/test/java/org/apache/flume/source/http/FlumeHttpServletRequestWrapper.java
>  475d92f 
>   
> flume-ng-core/src/test/java/org/apache/flume/source/http/TestBLOBHandler.java 
> f770d51 
>   
> flume-ng-core/src/test/java/org/apache/flume/source/http/TestHTTPSource.java 
> 3ad8282 
>   flume-ng-legacy-sources/flume-avro-source/pom.xml 7cecde2 
>   flume-ng-legacy-sources/flume-thrift-source/pom.xml 1ceeb5b 
>   flume-ng-sources/flume-scribe-source/pom.xml 73641df 
>   pom.xml f62c99a 
> 
> Diff: https://reviews.apache.org/r/52955/diff/
> 
> 
> Testing
> -------
> 
> Passes existing unit tests, further tests to follow.
> 
> 
> Thanks,
> 
> Tristan Stevens
> 
>

Reply via email to