----------------------------------------------------------- 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 > >