----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52955/ -----------------------------------------------------------
(Updated June 28, 2017, 12:55 p.m.) Review request for Flume. Changes ------- Update for JDK1.8 and Jetty 9.4 changes. 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. Finally, registered Jetty with the MBeans server to allow fine-grained performance tuning/monitoring if required. Diffs (updated) ----- flume-ng-core/pom.xml 2262d94f flume-ng-core/src/main/java/org/apache/flume/instrumentation/http/HTTPMetricsServer.java 921a1f74 flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSource.java 38bdfda6 flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSourceConfigurationConstants.java 86caf7d7 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 479cfc46 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 09d419f4 flume-ng-core/src/test/java/org/apache/flume/source/http/FlumeHttpServletRequestWrapper.java 475d92fe flume-ng-core/src/test/java/org/apache/flume/source/http/TestBLOBHandler.java f770d511 flume-ng-core/src/test/java/org/apache/flume/source/http/TestHTTPSource.java 3ad82829 flume-ng-doc/sphinx/FlumeUserGuide.rst 2073bf63 flume-ng-legacy-sources/flume-avro-source/pom.xml 7cecde22 flume-ng-legacy-sources/flume-thrift-source/pom.xml 1ceeb5b5 flume-ng-sources/flume-scribe-source/pom.xml 73641dfd pom.xml 5730db0c Diff: https://reviews.apache.org/r/52955/diff/3/ Changes: https://reviews.apache.org/r/52955/diff/2-3/ Testing ------- Passes existing unit tests, added specific tests for configurability aspects and also for JMX monitoring. Thanks, Tristan Stevens
