This is an automated email from the ASF dual-hosted git repository.
ctubbsii pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo-classloaders.git
The following commit(s) were added to refs/heads/main by this push:
new 9514fe4 Bump org.apache.commons:commons-vfs2 from 2.9.0 to 2.10.0
(#24)
9514fe4 is described below
commit 9514fe485ee4cb598397588c2fb327c9a60f5644
Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
AuthorDate: Tue Mar 25 14:04:22 2025 -0400
Bump org.apache.commons:commons-vfs2 from 2.9.0 to 2.10.0 (#24)
* Bump org.apache.commons:commons-vfs2 from 2.9.0 to 2.10.0
Bumps org.apache.commons:commons-vfs2 from 2.9.0 to 2.10.0.
---
updated-dependencies:
- dependency-name: org.apache.commons:commons-vfs2
dependency-type: direct:production
...
Signed-off-by: dependabot[bot] <[email protected]>
* Added changes to support vfs 2.10.0 release
Moved off of deprecated classes and methods.
Included new module that contained existing hdfs providers.
* Exclude hadoop deps and add http5 dep
Exclude the 3.4 hadoop deps from vfs2-hdfs.
Add the http5 client deps needed for the http5 provider
* Use BOM and fix compilation error
---------
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot]
<49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Daniel Roberts ddanielr <[email protected]>
Co-authored-by: Christopher Tubbs <[email protected]>
---
modules/vfs-class-loader/pom.xml | 28 ++++++++++++++++++++++
.../accumulo/classloader/vfs/VFSManager.java | 4 ++--
.../accumulo/classloader/vfs/AccumuloDFSBase.java | 4 ++--
pom.xml | 16 ++++++++++---
4 files changed, 45 insertions(+), 7 deletions(-)
diff --git a/modules/vfs-class-loader/pom.xml b/modules/vfs-class-loader/pom.xml
index edb8337..59c5bb7 100644
--- a/modules/vfs-class-loader/pom.xml
+++ b/modules/vfs-class-loader/pom.xml
@@ -88,6 +88,34 @@
<artifactId>slf4j-api</artifactId>
<scope>provided</scope>
</dependency>
+ <dependency>
+ <!-- no longer provided by accumulo after 2.1 -->
+ <groupId>org.apache.commons</groupId>
+ <artifactId>commons-vfs2-hdfs</artifactId>
+ <scope>runtime</scope>
+ <exclusions>
+ <exclusion>
+ <!-- exclude this outdated hadoop client from vfs -->
+ <groupId>org.apache.hadoop</groupId>
+ <artifactId>hadoop-common</artifactId>
+ </exclusion>
+ <exclusion>
+ <!-- exclude this outdated hadoop client from vfs -->
+ <groupId>org.apache.hadoop</groupId>
+ <artifactId>hadoop-hdfs</artifactId>
+ </exclusion>
+ <exclusion>
+ <!-- exclude this outdated hadoop client from vfs -->
+ <groupId>org.apache.hadoop</groupId>
+ <artifactId>hadoop-hdfs-client</artifactId>
+ </exclusion>
+ </exclusions>
+ </dependency>
+ <dependency>
+ <groupId>org.apache.httpcomponents.client5</groupId>
+ <artifactId>httpclient5</artifactId>
+ <scope>runtime</scope>
+ </dependency>
<dependency>
<groupId>org.apache.hadoop</groupId>
<artifactId>hadoop-client-minicluster</artifactId>
diff --git
a/modules/vfs-class-loader/src/main/java/org/apache/accumulo/classloader/vfs/VFSManager.java
b/modules/vfs-class-loader/src/main/java/org/apache/accumulo/classloader/vfs/VFSManager.java
index 09b27d9..fde94c8 100644
---
a/modules/vfs-class-loader/src/main/java/org/apache/accumulo/classloader/vfs/VFSManager.java
+++
b/modules/vfs-class-loader/src/main/java/org/apache/accumulo/classloader/vfs/VFSManager.java
@@ -154,8 +154,8 @@ public class VFSManager {
VFS.addProvider("file",
new
org.apache.commons.vfs2.provider.local.DefaultLocalFileProvider());
VFS.addProvider("jar", new
org.apache.commons.vfs2.provider.jar.JarFileProvider());
- VFS.addProvider("http", new
org.apache.commons.vfs2.provider.http.HttpFileProvider());
- VFS.addProvider("https", new
org.apache.commons.vfs2.provider.https.HttpsFileProvider());
+ VFS.addProvider("http", new
org.apache.commons.vfs2.provider.http5.Http5FileProvider());
+ VFS.addProvider("https", new
org.apache.commons.vfs2.provider.http5s.Http5sFileProvider());
VFS.addProvider("ftp", new
org.apache.commons.vfs2.provider.ftp.FtpFileProvider());
VFS.addProvider("ftps", new
org.apache.commons.vfs2.provider.ftps.FtpsFileProvider());
VFS.addProvider("war", new
org.apache.commons.vfs2.provider.jar.JarFileProvider());
diff --git
a/modules/vfs-class-loader/src/test/java/org/apache/accumulo/classloader/vfs/AccumuloDFSBase.java
b/modules/vfs-class-loader/src/test/java/org/apache/accumulo/classloader/vfs/AccumuloDFSBase.java
index ac41f3b..f82a521 100644
---
a/modules/vfs-class-loader/src/test/java/org/apache/accumulo/classloader/vfs/AccumuloDFSBase.java
+++
b/modules/vfs-class-loader/src/test/java/org/apache/accumulo/classloader/vfs/AccumuloDFSBase.java
@@ -114,8 +114,8 @@ public class AccumuloDFSBase {
vfs.addProvider("file",
new
org.apache.commons.vfs2.provider.local.DefaultLocalFileProvider());
vfs.addProvider("jar", new
org.apache.commons.vfs2.provider.jar.JarFileProvider());
- vfs.addProvider("http", new
org.apache.commons.vfs2.provider.http.HttpFileProvider());
- vfs.addProvider("https", new
org.apache.commons.vfs2.provider.https.HttpsFileProvider());
+ vfs.addProvider("http", new
org.apache.commons.vfs2.provider.http5.Http5FileProvider());
+ vfs.addProvider("https", new
org.apache.commons.vfs2.provider.http5s.Http5sFileProvider());
vfs.addProvider("ftp", new
org.apache.commons.vfs2.provider.ftp.FtpFileProvider());
vfs.addProvider("ftps", new
org.apache.commons.vfs2.provider.ftps.FtpsFileProvider());
vfs.addProvider("war", new
org.apache.commons.vfs2.provider.jar.JarFileProvider());
diff --git a/pom.xml b/pom.xml
index abb017e..198ee28 100644
--- a/pom.xml
+++ b/pom.xml
@@ -135,8 +135,15 @@ under the License.]]></licenseText>
<dependency>
<!-- no longer provided by accumulo after 2.1 -->
<groupId>org.apache.commons</groupId>
- <artifactId>commons-vfs2</artifactId>
- <version>2.9.0</version>
+ <artifactId>commons-vfs2-bom</artifactId>
+ <version>2.10.0</version>
+ <type>pom</type>
+ <scope>import</scope>
+ </dependency>
+ <dependency>
+ <groupId>org.apache.httpcomponents.client5</groupId>
+ <artifactId>httpclient5</artifactId>
+ <version>5.4.2</version>
</dependency>
</dependencies>
</dependencyManagement>
@@ -495,7 +502,10 @@ under the License.]]></licenseText>
<configuration>
<failOnWarning>true</failOnWarning>
<ignoredUnusedDeclaredDependencies>
-
<unused>org.apache.logging.log4j:log4j-slf4j2-impl:jar:*</unused>
+ <!-- ignore false positive runtime dependencies -->
+ <unused>org.apache.commons:commons-vfs2-hdfs:*</unused>
+
<unused>org.apache.httpcomponents.client5:httpclient5:*</unused>
+ <unused>org.apache.logging.log4j:log4j-slf4j2-impl:*</unused>
</ignoredUnusedDeclaredDependencies>
</configuration>
</execution>