>From Hussain Towaileb <[email protected]>:

Hussain Towaileb has uploaded this change for review. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21078?usp=email )


Change subject: [NO ISSUE][EXT]: propery check username and password presence
......................................................................

[NO ISSUE][EXT]: propery check username and password presence

Ext-ref: MB-71219
Change-Id: I4205f81a8e7810f82522dec0f926901b7dbe0dc9
---
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/iceberg/rest/RestUtils.java
1 file changed, 1 insertion(+), 1 deletion(-)



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

diff --git 
a/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/iceberg/rest/RestUtils.java
 
b/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/iceberg/rest/RestUtils.java
index ecbcf0d..602ba82 100644
--- 
a/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/iceberg/rest/RestUtils.java
+++ 
b/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/iceberg/rest/RestUtils.java
@@ -142,7 +142,7 @@
     private static void setBasicProperties(Map<String, String> 
catalogProperties) throws CompilationException {
         String username = catalogProperties.get(USERNAME_FIELD_NAME);
         String password = catalogProperties.get(PASSWORD_FIELD_NAME);
-        if (password != null) {
+        if (password == null) {
             throw 
CompilationException.create(REQUIRED_PARAM_IF_PARAM_IS_PRESENT, 
PASSWORD_FIELD_NAME,
                     USERNAME_FIELD_NAME);
         }

--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21078?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings?usp=email

Gerrit-MessageType: newchange
Gerrit-Project: asterixdb
Gerrit-Branch: lumina
Gerrit-Change-Id: I4205f81a8e7810f82522dec0f926901b7dbe0dc9
Gerrit-Change-Number: 21078
Gerrit-PatchSet: 1
Gerrit-Owner: Hussain Towaileb <[email protected]>

Reply via email to