This is an automated email from the ASF dual-hosted git repository.

ppkarwasz pushed a commit to branch drop-jetty-from-core
in repository https://gitbox.apache.org/repos/asf/logging-flume.git

commit 94c451110b1dffb56672e558a2a1d2dc63441dcf
Author: Piotr P. Karwasz <[email protected]>
AuthorDate: Wed Jun 10 15:13:46 2026 +0200

    Drop Jetty from flume-ng-core
    
    With the HTTP source and the HTTP/Prometheus monitors moved to their own
    modules, nothing in flume-ng-core's main code uses Jetty anymore. Remove
    the four jetty dependencies from the module.
    
    flume-ng-node's TestHttpConfigurationSource was relying on Jetty arriving
    transitively through flume-ng-core, so declare jetty-server and
    jetty-servlet as test dependencies there.
    
    Assisted-By: Claude Opus 4.8 (1M context) <[email protected]>
---
 flume-ng-core/pom.xml | 20 --------------------
 flume-ng-node/pom.xml | 12 ++++++++++++
 2 files changed, 12 insertions(+), 20 deletions(-)

diff --git a/flume-ng-core/pom.xml b/flume-ng-core/pom.xml
index 624b1188c..06a71baa1 100644
--- a/flume-ng-core/pom.xml
+++ b/flume-ng-core/pom.xml
@@ -131,26 +131,6 @@
       <artifactId>netty-all</artifactId>
     </dependency>
 
-    <dependency>
-      <groupId>org.eclipse.jetty</groupId>
-      <artifactId>jetty-servlet</artifactId>
-    </dependency>
-
-    <dependency>
-      <groupId>org.eclipse.jetty</groupId>
-      <artifactId>jetty-util</artifactId>
-    </dependency>
-
-    <dependency>
-      <groupId>org.eclipse.jetty</groupId>
-      <artifactId>jetty-server</artifactId>
-    </dependency>
-
-    <dependency>
-      <groupId>org.eclipse.jetty</groupId>
-      <artifactId>jetty-jmx</artifactId>
-    </dependency>
-
     <dependency>
       <groupId>org.apache.httpcomponents</groupId>
       <artifactId>httpclient</artifactId>
diff --git a/flume-ng-node/pom.xml b/flume-ng-node/pom.xml
index 2c2a6fea2..09349a6d8 100644
--- a/flume-ng-node/pom.xml
+++ b/flume-ng-node/pom.xml
@@ -140,6 +140,18 @@
       <scope>test</scope>
     </dependency>
 
+    <dependency>
+      <groupId>org.eclipse.jetty</groupId>
+      <artifactId>jetty-server</artifactId>
+      <scope>test</scope>
+    </dependency>
+
+    <dependency>
+      <groupId>org.eclipse.jetty</groupId>
+      <artifactId>jetty-servlet</artifactId>
+      <scope>test</scope>
+    </dependency>
+
     <dependency>
       <groupId>com.github.stefanbirkner</groupId>
       <artifactId>system-rules</artifactId>

Reply via email to