feat: remove hotpath layer and logging layer.

Author: Retrospection

bindings/java/Cargo.toml

@@ -50,7 +50,6 @@ default = [
   "layers-mime-guess",
   "layers-throttle",
   "layers-timeout",
-  "layers-hotpath",
 ]
 
 services-all = [
@@ -159,8 +158,6 @@ services-webdav = ["opendal/services-webdav"]
 services-webhdfs = ["opendal/services-webhdfs"]
 services-yandex-disk = ["opendal/services-yandex-disk"]
 
-layers-hotpath = ["opendal/layers-hotpath"]
-layers-logging = ["opendal/layers-logging"]
 layers-mime-guess = ["opendal/layers-mime-guess"]
 layers-throttle = ["opendal/layers-throttle"]
 layers-timeout = ["opendal/layers-timeout"]

bindings/java/src/layer.rs

@@ -24,7 +24,6 @@ use jni::sys::jfloat;
 use jni::sys::jlong;
 use opendal::Operator;
 use opendal::layers::ConcurrentLimitLayer;
-use opendal::layers::HotpathLayer;
 use opendal::layers::LoggingLayer;
 use opendal::layers::MimeGuessLayer;
 use opendal::layers::RetryLayer;
@@ -117,14 +116,3 @@ pub extern "system" fn Java_org_apache_opendal_layer_ThrottleLayer_doLayer(
     let throttle = ThrottleLayer::new(bandwidth as u32, burst as u32);
     Box::into_raw(Box::new(op.clone().layer(throttle))) as jlong
 }
-
-#[unsafe(no_mangle)]
-pub extern "system" fn Java_org_apache_opendal_layer_HotpathLayer_doLayer(
-    _: JNIEnv,
-    _: JClass,
-    op: *mut Operator,
-) -> jlong {
-    let op = unsafe { &*op };
-    let hotpath = HotpathLayer::new();
-    Box::into_raw(Box::new(op.clone().layer(hotpath))) as jlong
-}

bindings/java/src/main/java/org/apache/opendal/layer/HotpathLayer.java

@@ -24,6 +24,9 @@
 /**
  * This layer adds structured logging for every operation.
  *
+ * <p>Note: This layer requires proper logging setup (e.g., log4j2, java.util.logging) to work.
+ * Without logging configuration, this layer is a no-op.
+ *
  * @see <a href="https://docs.rs/opendal/latest/opendal/layers/struct.LoggingLayer.html">LoggingLayer's rustdoc</a>
  */
 public class LoggingLayer extends Layer {

bindings/java/src/main/java/org/apache/opendal/layer/LoggingLayer.java

@@ -25,7 +25,6 @@
 import org.apache.opendal.Layer;
 import org.apache.opendal.ServiceConfig;
 import org.apache.opendal.layer.ConcurrentLimitLayer;
-import org.apache.opendal.layer.HotpathLayer;
 import org.apache.opendal.layer.LoggingLayer;
 import org.apache.opendal.layer.MimeGuessLayer;
 import org.apache.opendal.layer.RetryLayer;
@@ -93,14 +92,4 @@ void testOperatorWithThrottleLayer() {
         @Cleanup final AsyncOperator layeredOp = op.layer(throttleLayer);
         assertThat(layeredOp.info).isNotNull();
     }
-
-    @Test
-    void testOperatorWithHotpathLayer() {
-        final ServiceConfig.Memory memory =
-                ServiceConfig.Memory.builder().root("/opendal/").build();
-        final Layer hotpathLayer = new HotpathLayer();
-        @Cleanup final AsyncOperator op = AsyncOperator.of(memory);
-        @Cleanup final AsyncOperator layeredOp = op.layer(hotpathLayer);
-        assertThat(layeredOp.info).isNotNull();
-    }
 }