----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52955/ -----------------------------------------------------------
(Updated Aug. 24, 2017, 8:32 p.m.) Review request for Flume. Changes ------- Fix checkstyle error Repository: flume-git Description ------- This patch upgrades the HTTPSource and HTTP Metrics server to Jetty 9.4. 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 e2ecd6ee 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/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 209db481 flume-ng-legacy-sources/flume-avro-source/pom.xml 46d301b5 flume-ng-legacy-sources/flume-thrift-source/pom.xml cb50473d flume-ng-sinks/flume-http-sink/pom.xml 3f8cfc88 flume-ng-sinks/flume-ng-morphline-solr-sink/src/test/java/org/apache/flume/sink/solr/morphline/FlumeHttpServletRequestWrapper.java 9711a3ad flume-ng-sources/flume-scribe-source/pom.xml 75fc085f pom.xml c0bd052b Diff: https://reviews.apache.org/r/52955/diff/6/ Changes: https://reviews.apache.org/r/52955/diff/5-6/ Testing ------- Passes existing unit tests, added specific tests for configurability aspects and also for JMX monitoring. File Attachments ---------------- FLUME-2698-4.patch https://reviews.apache.org/media/uploaded/files/2017/07/17/b0a55da7-1b6a-4ae1-aea5-d6efa7a43c69__FLUME-2698-4.patch Thanks, Tristan Stevens
