feat(storage): allow minio to use aws sdk or opendal

src/object_store/src/object/mod.rs

@@ -910,11 +910,21 @@ pub async fn build_remote_object_store(
             set your endpoint to the environment variable RW_S3_ENDPOINT.");
             panic!("Passing s3-compatible is not supported, please modify the environment variable and pass in s3.");
         }
-        minio if minio.starts_with("minio://") => ObjectStoreImpl::S3(
-            S3ObjectStore::with_minio(minio, metrics.clone(), config)
-                .await
-                .monitored(metrics),
-        ),
+        minio if minio.starts_with("minio://") => {
+            if std::env::var("RW_USE_OPENDAL_FOR_MINIO").is_ok() {
+                ObjectStoreImpl::Opendal(
+                    OpendalObjectStore::with_minio(minio)
+                        .unwrap()
+                        .monitored(metrics),
+                )
+            } else {
+                ObjectStoreImpl::S3(
+                    S3ObjectStore::with_minio(minio, metrics.clone(), config)
+                        .await
+                        .monitored(metrics),
+                )
+            }
+        }
         "memory" => {
             if ident == "Meta Backup" {
                 tracing::warn!("You're using in-memory remote object store for {}. This is not recommended for production environment.", ident);

src/object_store/src/object/opendal_engine/opendal_object_store.rs

@@ -38,6 +38,7 @@ pub enum EngineType {
     Memory,
     Hdfs,
     Gcs,
+    Minio,
     S3,
     Obs,
     Oss,
@@ -191,6 +192,7 @@ impl ObjectStore for OpendalObjectStore {
         match self.engine_type {
             EngineType::Memory => "Memory",
             EngineType::Hdfs => "Hdfs",
+            EngineType::Minio => "Minio",
             EngineType::S3 => "S3",
             EngineType::Gcs => "Gcs",
             EngineType::Obs => "Obs",

src/object_store/src/object/opendal_engine/opendal_s3.rs

@@ -39,7 +39,6 @@ impl OpendalObjectStore {
         if std::env::var("RW_IS_FORCE_PATH_STYLE").is_err() {
             builder.enable_virtual_host_style();
         }
-
         let op: Operator = Operator::new(builder)?
             .layer(LoggingLayer::default())
             .layer(
@@ -60,4 +59,38 @@ impl OpendalObjectStore {
             engine_type: EngineType::S3,
         })
     }
+
+    /// Creates a minio client. The server should be like `minio://key:secret@address:port/bucket`.
+    pub fn with_minio(server: &str) -> ObjectResult<Self> {
+        let server = server.strip_prefix("minio://").unwrap();
+        let (access_key_id, rest) = server.split_once(':').unwrap();
+        let (secret_access_key, mut rest) = rest.split_once('@').unwrap();
+        let endpoint_prefix = if let Some(rest_stripped) = rest.strip_prefix("https://") {
+            rest = rest_stripped;
+            "https://"
+        } else if let Some(rest_stripped) = rest.strip_prefix("http://") {
+            rest = rest_stripped;
+            "http://"
+        } else {
+            "http://"
+        };
+        let (address, bucket) = rest.split_once('/').unwrap();
+        let mut builder = S3::default();
+        builder
+            .bucket(bucket)
+            .region("custom")
+            .access_key_id(access_key_id)
+            .secret_access_key(secret_access_key)
+            .endpoint(&format!("{}{}", endpoint_prefix, address));
+
+        builder.disable_config_load();
+        let op: Operator = Operator::new(builder)?
+            .layer(LoggingLayer::default())
+            .layer(RetryLayer::default())
+            .finish();
+        Ok(Self {
+            op,
+            engine_type: EngineType::Minio,
+        })
+    }
 }

src/object_store/src/object/s3.rs

@@ -62,13 +62,13 @@ type PartId = i32;
 
 /// MinIO and S3 share the same minimum part ID and part size.
 const MIN_PART_ID: PartId = 1;
+// TODO: we should do some benchmark to determine the proper part size for MinIO
+const MINIO_PART_SIZE: usize = 16 * 1024 * 1024;
 /// The minimum number of bytes that is buffered before they are uploaded as a part.
 /// Its value must be greater than the minimum part size of 5MiB.
 ///
 /// Reference: <https://docs.aws.amazon.com/AmazonS3/latest/userguide/qfacts.html>
 const S3_PART_SIZE: usize = 16 * 1024 * 1024;
-// TODO: we should do some benchmark to determine the proper part size for MinIO
-const MINIO_PART_SIZE: usize = 16 * 1024 * 1024;
 /// The number of S3/MinIO bucket prefixes
 const NUM_BUCKET_PREFIXES: u32 = 256;
 /// Stop multipart uploads that don't complete within a specified number of days after being

src/risedevtool/src/task/utils.rs

@@ -118,6 +118,10 @@ pub fn add_hummock_backend(
                 cmd.arg("--state-store")
                 .arg(format!("hummock+hdfs://{}", opendal.namenode));
             }
+            else if opendal.engine == "opendal_s3"{
+                cmd.arg("--state-store")
+                .arg(format!("hummock+opendal_s3://{}", opendal.bucket));
+            }
             else if opendal.engine == "gcs"{
                 cmd.arg("--state-store")
                 .arg(format!("hummock+gcs://{}", opendal.bucket));