Change in asterixdb[master]: [ASTERIXDB-3456][EXT] Allow S3 endpoint to be null

[AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu>]

From Murtadha Hubail <mh...@apache.org>:

Murtadha Hubail has uploaded this change for review. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18467 )


Change subject: [ASTERIXDB-3456][EXT] Allow S3 endpoint to be null
......................................................................

[ASTERIXDB-3456][EXT] Allow S3 endpoint to be null

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

Details:

- Endpoint is optional and can be null for S3 clients.

Ext-ref: MB-62688
Change-Id: I78c2d2dd5da166b32e86c74fe38d66a97d321852
---
M asterixdb/asterix-cloud/src/main/java/org/apache/asterix/cloud/clients/aws/s3/S3ClientConfig.java
1 file changed, 19 insertions(+), 1 deletion(-)



  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/67/18467/1

diff --git a/asterixdb/asterix-cloud/src/main/java/org/apache/asterix/cloud/clients/aws/s3/S3ClientConfig.java b/asterixdb/asterix-cloud/src/main/java/org/apache/asterix/cloud/clients/aws/s3/S3ClientConfig.java
index 37872dd..fe73a0a 100644
--- a/asterixdb/asterix-cloud/src/main/java/org/apache/asterix/cloud/clients/aws/s3/S3ClientConfig.java
+++ b/asterixdb/asterix-cloud/src/main/java/org/apache/asterix/cloud/clients/aws/s3/S3ClientConfig.java
@@ -42,7 +42,7 @@
     public S3ClientConfig(String region, String endpoint, String prefix, boolean anonymousAuth,
             long profilerLogInterval, int writeBufferSize) {
         this.region = Objects.requireNonNull(region, "region");
-        this.endpoint = Objects.requireNonNull(endpoint, "endpoint");
+        this.endpoint = endpoint;
         this.prefix = Objects.requireNonNull(prefix, "prefix");
         this.anonymousAuth = anonymousAuth;
         this.profilerLogInterval = profilerLogInterval;

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18467
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: I78c2d2dd5da166b32e86c74fe38d66a97d321852
Gerrit-Change-Number: 18467
Gerrit-PatchSet: 1
Gerrit-Owner: Murtadha Hubail <mh...@apache.org>
Gerrit-MessageType: newchange