----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52955/ -----------------------------------------------------------
(Updated Oct. 18, 2016, 4:03 p.m.) Review request for Flume. Changes ------- Added tests, JMX and changed reference to released stable version of Jetty 9.2.19.v20160908. This diff should now be stable and passes tests and checkstyle. Repository: flume-git Description (updated) ------- 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. Finally, registered Jetty with the MBeans server to allow fine-grained performance tuning/monitoring if required. Diffs (updated) ----- flume-ng-core/pom.xml 2262d94 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/main/java/org/apache/flume/tools/ImmutablePair.java PRE-CREATION 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-doc/sphinx/FlumeUserGuide.rst cd76bb3 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 (updated) ------- Passes existing unit tests, added specific tests for configurability aspects and also for JMX monitoring. Thanks, Tristan Stevens