From cccda183a2086644a351865751e0fe2dca65e1eb Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Wed, 6 Nov 2024 15:20:30 +0800 Subject: [PATCH] [#5473] improvement(CI): Change CI build file to add bundles files into the build task. (#5485) ### 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. Co-authored-by: Qi Yu --- .github/workflows/build.yml | 1 + .../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 8490b4d851f..b36db4ccfcb 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 b61e9d14f60..0d755c1f564 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 FileSystem getFileSystem(Path path, Map config) throws IO 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";