Issue #5086 Review and rework o.e.j.util.Scanner

Signed-off-by: Jan Bartel <janb@webtide.com>
jetty · Dec 1, 2020 · 4883332 · 4883332
1 parent 7b77a53
commit 4883332
Show file tree

Hide file tree

Showing 8 changed files with 273 additions and 330 deletions.
diff --git a/jetty-ant/src/main/java/org/eclipse/jetty/ant/ServerProxyImpl.java b/jetty-ant/src/main/java/org/eclipse/jetty/ant/ServerProxyImpl.java
@@ -24,6 +24,7 @@
 import java.util.ArrayList;
 import java.util.Iterator;
 import java.util.List;
+import java.util.Set;
 
 import org.eclipse.jetty.ant.types.Connector;
 import org.eclipse.jetty.ant.types.ContextHandlers;
@@ -135,7 +136,7 @@ public WebAppScannerListener(AntWebAppContext awc)
         }
 
         @Override
-        public void filesChanged(List<String> changedFileNames)
+        public void filesChanged(Set<String> changedFileNames)
         {
             boolean isScanned = false;
             try

diff --git a/jetty-deploy/src/main/java/org/eclipse/jetty/deploy/providers/ScanningAppProvider.java b/jetty-deploy/src/main/java/org/eclipse/jetty/deploy/providers/ScanningAppProvider.java
@@ -140,7 +140,6 @@ protected void doStart() throws Exception
         _scanner = new Scanner();
         _scanner.setScanDirs(files);
         _scanner.setScanInterval(_scanInterval);
-        _scanner.setRecursive(_recursive);
         _scanner.setFilenameFilter(_filenameFilter);
         _scanner.setReportDirs(true);
         _scanner.setScanDepth(1); //consider direct dir children of monitored dir

diff --git a/...c/test/java/org/eclipse/jetty/deploy/providers/ScanningAppProviderRuntimeUpdatesTest.java b/...c/test/java/org/eclipse/jetty/deploy/providers/ScanningAppProviderRuntimeUpdatesTest.java
@@ -76,10 +76,15 @@ public void setupEnvironment() throws Exception
             if (provider instanceof ScanningAppProvider)
             {
                 _providers++;
-                ((ScanningAppProvider)provider).addScannerListener(new Scanner.ScanListener()
+                ((ScanningAppProvider)provider).addScannerListener(new Scanner.ScanCycleListener()
                 {
                     @Override
-                    public void scan()
+                    public void scanStarted(int cycle)
+                    {
+                    }
+
+                    @Override
+                    public void scanEnded(int cycle)
                     {
                         _scans.incrementAndGet();
                     }

diff --git a/jetty-maven-plugin/src/main/java/org/eclipse/jetty/maven/plugin/JettyRunMojo.java b/jetty-maven-plugin/src/main/java/org/eclipse/jetty/maven/plugin/JettyRunMojo.java
@@ -22,7 +22,7 @@
 import java.nio.file.Path;
 import java.nio.file.PathMatcher;
 import java.util.Date;
-import java.util.List;
+import java.util.Set;
 
 import org.apache.maven.artifact.Artifact;
 import org.apache.maven.plugin.MojoExecutionException;
@@ -191,7 +191,7 @@ protected void configureScanner()
         }
         scanner.addListener(new Scanner.BulkListener()
         {
-            public void filesChanged(List<String> changes)
+            public void filesChanged(Set<String> changes)
             {
                 try
                 {

diff --git a/jetty-maven-plugin/src/main/java/org/eclipse/jetty/maven/plugin/JettyRunWarMojo.java b/jetty-maven-plugin/src/main/java/org/eclipse/jetty/maven/plugin/JettyRunWarMojo.java
@@ -22,7 +22,7 @@
 import java.nio.file.Path;
 import java.nio.file.Paths;
 import java.util.Date;
-import java.util.List;
+import java.util.Set;
 
 import org.apache.maven.plugin.MojoExecutionException;
 import org.apache.maven.plugins.annotations.Execute;
@@ -197,7 +197,7 @@ public void configureScanner() throws MojoExecutionException
             configureScanTargetPatterns(scanner);
             scanner.addListener(new Scanner.BulkListener()
             {
-                public void filesChanged(List<String> changes)
+                public void filesChanged(Set<String> changes)
                 {
                     try
                     {