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";