You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by xi...@apache.org on 2022/01/22 16:48:50 UTC
[pinot] branch master updated: Add error message if region is null or empty during creating S3PinotFS (#8056)
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 <wc...@gmail.com>
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 <ch...@cloudkitchens.com>
---
.../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: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org