>From Michael Blow <[email protected]>:

Attention is currently required from: Hussain Altowileb, Preetham Poluparthi.

Michael Blow has posted comments on this change by Preetham Poluparthi. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21277?usp=email )

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


Patch Set 6:

(1 comment)

File 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/aws/s3/S3Utils.java:

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21277/comment/6b7f977a_babb477c?usp=email
 :
PS6, Line 211:             
builder.serviceConfiguration(S3Configuration.builder().chunkedEncodingEnabled(false).build());
> We don't want to set chunked encoding implicitly. There's probably no harm in 
> `LegacyMd5Plugin. […]
Sync'd offline w/ Hussain on this- we should add first-class configuration 
parameters to S3 links for the legacy md5 checksums & disabling chunked 
encoding, as we have done for checksum behavior (i.e. 
`CHECKSUM_BEHAVIOR_FIELD_NAME`).



--
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: comment
Gerrit-Project: asterixdb
Gerrit-Branch: lumina
Gerrit-Change-Id: I9443c9327a6bedb7e39c2ef1b729962b2237a974
Gerrit-Change-Number: 21277
Gerrit-PatchSet: 6
Gerrit-Owner: Preetham Poluparthi <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-CC: Hussain Altowileb <[email protected]>
Gerrit-CC: Michael Blow <[email protected]>
Gerrit-Attention: Hussain Altowileb <[email protected]>
Gerrit-Attention: Preetham Poluparthi <[email protected]>
Gerrit-Comment-Date: Tue, 26 May 2026 12:41:37 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Michael Blow <[email protected]>

Reply via email to