This is an automated email from the ASF dual-hosted git repository.
xiangfu0 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 5c70d884960 Update the s3Config to take in globally from init (#18679)
5c70d884960 is described below
commit 5c70d884960a23801688e7484950cebc1ee948c3
Author: Chaitanya Deepthi <[email protected]>
AuthorDate: Thu Jun 4 01:34:32 2026 -0700
Update the s3Config to take in globally from init (#18679)
---
.../main/java/org/apache/pinot/plugin/filesystem/S3PinotFS.java | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git
a/pinot-plugins/pinot-file-system/pinot-s3/src/main/java/org/apache/pinot/plugin/filesystem/S3PinotFS.java
b/pinot-plugins/pinot-file-system/pinot-s3/src/main/java/org/apache/pinot/plugin/filesystem/S3PinotFS.java
index 10b18642f8c..f0d392d076a 100644
---
a/pinot-plugins/pinot-file-system/pinot-s3/src/main/java/org/apache/pinot/plugin/filesystem/S3PinotFS.java
+++
b/pinot-plugins/pinot-file-system/pinot-s3/src/main/java/org/apache/pinot/plugin/filesystem/S3PinotFS.java
@@ -316,10 +316,10 @@ public class S3PinotFS extends BasePinotFS {
*/
public void init(S3Client s3Client, String serverSideEncryption,
PinotConfiguration serverSideEncryptionConfig) {
_s3Client = s3Client;
- S3Config s3Config = new S3Config(serverSideEncryptionConfig);
- setServerSideEncryption(serverSideEncryption, s3Config);
- setMultiPartUploadConfigs(s3Config);
- setDisableAcl(s3Config);
+ _s3Config = new S3Config(serverSideEncryptionConfig);
+ setServerSideEncryption(serverSideEncryption, _s3Config);
+ setMultiPartUploadConfigs(_s3Config);
+ setDisableAcl(_s3Config);
}
@VisibleForTesting
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
