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

jackie 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 dfd8b1bd2b Upgrade AWS SDK to 2.30; update tests to disable default 
integrity protection (#14909)
dfd8b1bd2b is described below

commit dfd8b1bd2b41e42049df4562d8f2f3b3165d5dcc
Author: Yash Mayya <[email protected]>
AuthorDate: Fri Jan 24 12:33:37 2025 +0700

    Upgrade AWS SDK to 2.30; update tests to disable default integrity 
protection (#14909)
---
 .../java/org/apache/pinot/plugin/filesystem/S3PinotFSTest.java     | 7 ++++++-
 pom.xml                                                            | 2 +-
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git 
a/pinot-plugins/pinot-file-system/pinot-s3/src/test/java/org/apache/pinot/plugin/filesystem/S3PinotFSTest.java
 
b/pinot-plugins/pinot-file-system/pinot-s3/src/test/java/org/apache/pinot/plugin/filesystem/S3PinotFSTest.java
index fd3f8b2bd7..8c9ed7f5fe 100644
--- 
a/pinot-plugins/pinot-file-system/pinot-s3/src/test/java/org/apache/pinot/plugin/filesystem/S3PinotFSTest.java
+++ 
b/pinot-plugins/pinot-file-system/pinot-s3/src/test/java/org/apache/pinot/plugin/filesystem/S3PinotFSTest.java
@@ -41,6 +41,8 @@ import org.testng.annotations.DataProvider;
 import org.testng.annotations.Test;
 import software.amazon.awssdk.auth.credentials.AwsBasicCredentials;
 import software.amazon.awssdk.auth.credentials.StaticCredentialsProvider;
+import software.amazon.awssdk.core.checksums.RequestChecksumCalculation;
+import software.amazon.awssdk.core.checksums.ResponseChecksumValidation;
 import software.amazon.awssdk.core.sync.RequestBody;
 import software.amazon.awssdk.regions.Region;
 import software.amazon.awssdk.services.s3.S3Client;
@@ -527,6 +529,9 @@ public class S3PinotFSTest {
     return S3Client.builder().region(Region.of("us-east-1"))
         
.credentialsProvider(StaticCredentialsProvider.create(AwsBasicCredentials.create("foo",
 "bar")))
         
.serviceConfiguration(S3Configuration.builder().pathStyleAccessEnabled(true).build())
-        .endpointOverride(URI.create(endpoint)).build();
+        .endpointOverride(URI.create(endpoint))
+        .responseChecksumValidation(ResponseChecksumValidation.WHEN_REQUIRED)
+        .requestChecksumCalculation(RequestChecksumCalculation.WHEN_REQUIRED)
+        .build();
   }
 }
diff --git a/pom.xml b/pom.xml
index 2d5a3878ea..512c9a07d0 100644
--- a/pom.xml
+++ b/pom.xml
@@ -175,7 +175,7 @@
     <audienceannotations.version>0.15.0</audienceannotations.version>
     <clp-ffi.version>0.4.7</clp-ffi.version>
     <stax2-api.version>4.2.2</stax2-api.version>
-    <aws.sdk.version>2.29.52</aws.sdk.version>
+    <aws.sdk.version>2.30.4</aws.sdk.version>
     <azure.sdk.version>1.2.30</azure.sdk.version>
     <azure.msal4j.version>1.18.0</azure.msal4j.version>
     <joda-time.version>2.13.0</joda-time.version>


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to