>From Preetham Poluparthi <[email protected]>:

Attention is currently required from: Preetham Poluparthi.

Hello Anon. E. Moose #1000171, Jenkins,

I'd like you to reexamine a change. Please visit

    https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21277?usp=email

to look at the new patch set (#4).

The following approvals got outdated and were removed:
Contrib+1 by Anon. E. Moose #1000171, Integration-Tests+1 by Jenkins, 
Verified-1 by Jenkins


Change subject: [ASTERIXDB-3768][CLOUD] Add S3 legacy MD5 and disable chunked 
encoding knobs
......................................................................

[ASTERIXDB-3768][CLOUD] Add S3 legacy MD5 and disable chunked encoding knobs

- user model changes: no
- storage format changes: no
- interface changes: no

 Add isS3LegacyMd5ChecksumEnabled() and isS3ChunkedEncodingDisabled() to 
ICloudProperties with endpoint-aware defaults in CloudProperties (true when a 
custom endpoint is set, false for native AWS S3). S3ClientConfig carries both 
flags; S3CloudClient applies them independently of checksum behavior. S3Utils 
(external data path) applies LegacyMd5Plugin and disables chunked encoding when 
WHEN_REQUIRED is resolved, keeping the behavior tied to the per-link checksum 
setting for that path.

Ext-ref: MB-72035

Change-Id: I9443c9327a6bedb7e39c2ef1b729962b2237a974
---
M 
asterixdb/asterix-cloud/src/main/java/org/apache/asterix/cloud/clients/aws/s3/S3ClientConfig.java
M 
asterixdb/asterix-cloud/src/main/java/org/apache/asterix/cloud/clients/aws/s3/S3CloudClient.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CloudProperties.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/aws/s3/S3Utils.java
M 
hyracks-fullstack/hyracks/hyracks-cloud/src/main/java/org/apache/hyracks/cloud/io/ICloudProperties.java
5 files changed, 67 insertions(+), 4 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/77/21277/4
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21277?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings?usp=email

Gerrit-MessageType: newpatchset
Gerrit-Project: asterixdb
Gerrit-Branch: lumina
Gerrit-Change-Id: I9443c9327a6bedb7e39c2ef1b729962b2237a974
Gerrit-Change-Number: 21277
Gerrit-PatchSet: 4
Gerrit-Owner: Preetham Poluparthi <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-CC: Michael Blow <[email protected]>
Gerrit-Attention: Preetham Poluparthi <[email protected]>

Reply via email to