This is an automated email from the ASF dual-hosted git repository.
xiangfu 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 e23d5e7 Add error message if region is null or empty during creating
S3PinotFS (#8056)
e23d5e7 is described below
commit e23d5e7b80a5dfaa44b7dc00f2548da24b500680
Author: WangCHX <[email protected]>
AuthorDate: Sun Jan 23 00:48:22 2022 +0800
Add error message if region is null or empty during creating S3PinotFS
(#8056)
Co-authored-by: chengxuan.wang <[email protected]>
---
.../src/main/java/org/apache/pinot/plugin/filesystem/S3PinotFS.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/pinot-plugins/pinot-file-system/pinot-s3/src/main/java/org/apache/pinot/plugin/filesystem/S3PinotFS.java
b/pinot-plugins/pinot-file-system/pinot-s3/src/main/java/org/apache/pinot/plugin/filesystem/S3PinotFS.java
index 0797cc7..d4d4aa7 100644
---
a/pinot-plugins/pinot-file-system/pinot-s3/src/main/java/org/apache/pinot/plugin/filesystem/S3PinotFS.java
+++
b/pinot-plugins/pinot-file-system/pinot-s3/src/main/java/org/apache/pinot/plugin/filesystem/S3PinotFS.java
@@ -92,7 +92,7 @@ public class S3PinotFS extends PinotFS {
@Override
public void init(PinotConfiguration config) {
- Preconditions.checkArgument(!isNullOrEmpty(config.getProperty(REGION)));
+ Preconditions.checkArgument(!isNullOrEmpty(config.getProperty(REGION)),
"Region can't be null or empty");
String region = config.getProperty(REGION);
_disableAcl = config.getProperty(DISABLE_ACL_CONFIG_KEY,
DEFAULT_DISABLE_ACL);
String serverSideEncryption =
config.getProperty(SERVER_SIDE_ENCRYPTION_CONFIG_KEY);
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
