Skip to content

Commit cccda18

Browse files
[#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 <yuqi@datastrato.com>
1 parent 81b50c5 commit cccda18

File tree

2 files changed

+5
-0
lines changed

2 files changed

+5
-0
lines changed

.github/workflows/build.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@ jobs:
3737
- core/**
3838
- dev/**
3939
- docs/open-api/**
40+
- bundles/**
4041
- docs/build.gradle.kts
4142
- flink-connector/**
4243
- gradle/**

bundles/aws-bundle/src/main/java/org/apache/gravitino/s3/fs/S3FileSystemProvider.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,10 @@ public FileSystem getFileSystem(Path path, Map<String, String> config) throws IO
5555
return S3AFileSystem.newInstance(path.toUri(), configuration);
5656
}
5757

58+
/**
59+
* Get the scheme of the FileSystem. Attention, for S3 the schema is "s3a", not "s3". Users should
60+
* use "s3a://..." to access S3.
61+
*/
5862
@Override
5963
public String scheme() {
6064
return "s3a";

0 commit comments

Comments
 (0)