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 c492250 adjusts logging (#54)
c492250 is described below
commit c492250e9a8275982ecc954acdb3c1212624a232
Author: Keith Turner <[email protected]>
AuthorDate: Wed Jan 28 17:50:02 2026 -0800
adjusts logging (#54)
Limited downloads to a single debug message. Promoted classloader
creation to info.
Removed unused import for Arrays in LccUtils.java
---------
Co-authored-by: Christopher Tubbs <[email protected]>
---
.../hdfsurlstreamhandlerprovider/HdfsURLStreamHandlerProvider.java | 2 +-
.../java/org/apache/accumulo/classloader/lcc/util/LccUtils.java | 6 +-----
.../java/org/apache/accumulo/classloader/lcc/util/LocalStore.java | 2 +-
3 files changed, 3 insertions(+), 7 deletions(-)
diff --git
a/modules/hdfs-urlstreamhandler-provider/src/main/java/org/apache/accumulo/classloader/hdfsurlstreamhandlerprovider/HdfsURLStreamHandlerProvider.java
b/modules/hdfs-urlstreamhandler-provider/src/main/java/org/apache/accumulo/classloader/hdfsurlstreamhandlerprovider/HdfsURLStreamHandlerProvider.java
index aff19e1..861d0f4 100644
---
a/modules/hdfs-urlstreamhandler-provider/src/main/java/org/apache/accumulo/classloader/hdfsurlstreamhandlerprovider/HdfsURLStreamHandlerProvider.java
+++
b/modules/hdfs-urlstreamhandler-provider/src/main/java/org/apache/accumulo/classloader/hdfsurlstreamhandlerprovider/HdfsURLStreamHandlerProvider.java
@@ -58,7 +58,7 @@ public class HdfsURLStreamHandlerProvider extends
URLStreamHandlerProvider {
@Override
public void connect() throws IOException {
Preconditions.checkState(is == null, "Already connected");
- LOG.debug("Connecting to {}", url);
+ LOG.trace("Connecting to {}", url);
final URI uri;
try {
uri = url.toURI();
diff --git
a/modules/local-caching-classloader/src/main/java/org/apache/accumulo/classloader/lcc/util/LccUtils.java
b/modules/local-caching-classloader/src/main/java/org/apache/accumulo/classloader/lcc/util/LccUtils.java
index e9c67f2..22a8f1d 100644
---
a/modules/local-caching-classloader/src/main/java/org/apache/accumulo/classloader/lcc/util/LccUtils.java
+++
b/modules/local-caching-classloader/src/main/java/org/apache/accumulo/classloader/lcc/util/LccUtils.java
@@ -20,7 +20,6 @@ package org.apache.accumulo.classloader.lcc.util;
import java.net.URL;
import java.net.URLClassLoader;
-import java.util.Arrays;
import
org.apache.accumulo.classloader.lcc.LocalCachingContextClassLoaderFactory;
import org.apache.commons.codec.digest.DigestUtils;
@@ -39,10 +38,7 @@ public class LccUtils {
public static URLClassLoader createClassLoader(String name, URL[] urls) {
final var cl = new URLClassLoader(name, urls,
LocalCachingContextClassLoaderFactory.class.getClassLoader());
- if (LOG.isTraceEnabled()) {
- LOG.trace("New classloader created for {} from URLs: {}", name,
Arrays.asList(urls));
- }
+ LOG.info("New classloader created for {}", name);
return cl;
}
-
}
diff --git
a/modules/local-caching-classloader/src/main/java/org/apache/accumulo/classloader/lcc/util/LocalStore.java
b/modules/local-caching-classloader/src/main/java/org/apache/accumulo/classloader/lcc/util/LocalStore.java
index 329fac4..9d533bb 100644
---
a/modules/local-caching-classloader/src/main/java/org/apache/accumulo/classloader/lcc/util/LocalStore.java
+++
b/modules/local-caching-classloader/src/main/java/org/apache/accumulo/classloader/lcc/util/LocalStore.java
@@ -145,7 +145,7 @@ public final class LocalStore {
for (Resource resource : contextDefinition.getResources()) {
Path path = storeResource(resource);
if (path == null) {
- LOG.debug("Skipped resource {} while another process or thread is
downloading it",
+ LOG.trace("Skipped resource {} while another process or thread is
downloading it",
resource.getLocation());
continue;
}