This is an automated email from the ASF dual-hosted git repository.

jshao pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/main by this push:
     new 06aa93987 [#5473] improvement(CI): Change CI build file to add bundles 
files into the build task. (#5479)
06aa93987 is described below

commit 06aa93987597d4e99c546f05e3780e74b9b78c40
Author: Qi Yu <[email protected]>
AuthorDate: Wed Nov 6 14:24:47 2024 +0800

    [#5473] improvement(CI): Change CI build file to add bundles files into the 
build task. (#5479)
    
    ### What changes were proposed in this pull request?
    
    
    Add the bundles and their sub-modules to the changes listening list.
    
    ### Why are the changes needed?
    
    Bundles are newly added modules and should be added to the changes
    listening list.
    
    Fix: #5473
    
    ### Does this PR introduce _any_ user-facing change?
    
    N/A
    
    ### How was this patch tested?
    
    CI.
---
 .github/workflows/build.yml                                           | 1 +
 .../main/java/org/apache/gravitino/s3/fs/S3FileSystemProvider.java    | 4 ++++
 2 files changed, 5 insertions(+)

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index 8490b4d85..b36db4ccf 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -37,6 +37,7 @@ jobs:
               - core/**
               - dev/**
               - docs/open-api/**
+              - bundles/**
               - docs/build.gradle.kts
               - flink-connector/**
               - gradle/**
diff --git 
a/bundles/aws-bundle/src/main/java/org/apache/gravitino/s3/fs/S3FileSystemProvider.java
 
b/bundles/aws-bundle/src/main/java/org/apache/gravitino/s3/fs/S3FileSystemProvider.java
index b61e9d14f..0d755c1f5 100644
--- 
a/bundles/aws-bundle/src/main/java/org/apache/gravitino/s3/fs/S3FileSystemProvider.java
+++ 
b/bundles/aws-bundle/src/main/java/org/apache/gravitino/s3/fs/S3FileSystemProvider.java
@@ -55,6 +55,10 @@ public class S3FileSystemProvider implements 
FileSystemProvider {
     return S3AFileSystem.newInstance(path.toUri(), configuration);
   }
 
+  /**
+   * Get the scheme of the FileSystem. Attention, for S3 the schema is "s3a", 
not "s3". Users should
+   * use "s3a://..." to access S3.
+   */
   @Override
   public String scheme() {
     return "s3a";

Reply via email to