>From Michael Blow <[email protected]>: Michael Blow has submitted this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19732 )
( 1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: [NO ISSUE][*DB][STO] Fix gate for cloud list eventually consistent check ...................................................................... [NO ISSUE][*DB][STO] Fix gate for cloud list eventually consistent check Correct inverted check to validate list results when the CLOUD_STORAGE_LIST_EVENTUALLY_CONSISTENT option is configured true Ext-ref: MB-66343 Change-Id: I561975e684a02a92cd05ddaadf88e1a4ee56c899 Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19732 Reviewed-by: Michael Blow <[email protected]> Reviewed-by: Ritik Raj <[email protected]> Tested-by: Michael Blow <[email protected]> --- M asterixdb/asterix-cloud/src/main/java/org/apache/asterix/cloud/clients/aws/s3/S3CloudClient.java 1 file changed, 18 insertions(+), 1 deletion(-) Approvals: Michael Blow: Looks good to me, but someone else must approve; Verified Ritik Raj: Looks good to me, approved diff --git a/asterixdb/asterix-cloud/src/main/java/org/apache/asterix/cloud/clients/aws/s3/S3CloudClient.java b/asterixdb/asterix-cloud/src/main/java/org/apache/asterix/cloud/clients/aws/s3/S3CloudClient.java index 77d34a2..fc35792 100644 --- a/asterixdb/asterix-cloud/src/main/java/org/apache/asterix/cloud/clients/aws/s3/S3CloudClient.java +++ b/asterixdb/asterix-cloud/src/main/java/org/apache/asterix/cloud/clients/aws/s3/S3CloudClient.java @@ -371,7 +371,7 @@ private <T, C extends Collection<T>> C ensureListConsistent(C cloudFiles, String bucket, Function<T, String> pathExtractor) { - if (config.isStorageListEventuallyConsistent()) { + if (!config.isStorageListEventuallyConsistent()) { return cloudFiles; } Iterator<T> iterator = cloudFiles.iterator(); -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19732 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: I561975e684a02a92cd05ddaadf88e1a4ee56c899 Gerrit-Change-Number: 19732 Gerrit-PatchSet: 3 Gerrit-Owner: Michael Blow <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Ritik Raj <[email protected]> Gerrit-CC: Anon. E. Moose #1000171 Gerrit-CC: Jenkins <[email protected]> Gerrit-MessageType: merged
