>From Hussain Towaileb <[email protected]>:

Hussain Towaileb has submitted this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19488 )

 (

6 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted 
one.
 )Change subject: [NO ISSUE][EXT]: Correctly pass session token property for 
hadoop
......................................................................

[NO ISSUE][EXT]: Correctly pass session token property for hadoop

Ext-ref: MB-65588
Change-Id: Idee6fb65900b50417f26be88975ef3b186728789
Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19488
Integration-Tests: Jenkins <[email protected]>
Tested-by: Jenkins <[email protected]>
Reviewed-by: Peeyush Gupta <[email protected]>
---
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/aws/s3/S3AuthUtils.java
1 file changed, 16 insertions(+), 1 deletion(-)

Approvals:
  Peeyush Gupta: Looks good to me, approved
  Jenkins: Verified; Verified




diff --git 
a/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/aws/s3/S3AuthUtils.java
 
b/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/aws/s3/S3AuthUtils.java
index 4e0a435..f6ca480 100644
--- 
a/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/aws/s3/S3AuthUtils.java
+++ 
b/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/aws/s3/S3AuthUtils.java
@@ -51,6 +51,7 @@
 import static 
org.apache.asterix.external.util.aws.s3.S3Constants.HADOOP_SERVICE_END_POINT;
 import static 
org.apache.asterix.external.util.aws.s3.S3Constants.HADOOP_SESSION_TOKEN;
 import static 
org.apache.asterix.external.util.aws.s3.S3Constants.HADOOP_SIMPLE;
+import static 
org.apache.asterix.external.util.aws.s3.S3Constants.HADOOP_TEMPORARY;
 import static 
org.apache.asterix.external.util.aws.s3.S3Constants.INSTANCE_PROFILE_FIELD_NAME;
 import static 
org.apache.asterix.external.util.aws.s3.S3Constants.REGION_FIELD_NAME;
 import static 
org.apache.asterix.external.util.aws.s3.S3Constants.ROLE_ARN_FIELD_NAME;
@@ -452,7 +453,7 @@
                 jobConf.set(HADOOP_ACCESS_KEY_ID, 
configuration.get(ACCESS_KEY_ID_FIELD_NAME));
                 jobConf.set(HADOOP_SECRET_ACCESS_KEY, 
configuration.get(SECRET_ACCESS_KEY_FIELD_NAME));
                 if (configuration.get(SESSION_TOKEN_FIELD_NAME) != null) {
-                    jobConf.set(HADOOP_CREDENTIAL_PROVIDER_KEY, 
HADOOP_SESSION_TOKEN);
+                    jobConf.set(HADOOP_CREDENTIAL_PROVIDER_KEY, 
HADOOP_TEMPORARY);
                     jobConf.set(HADOOP_SESSION_TOKEN, 
configuration.get(SESSION_TOKEN_FIELD_NAME));
                 }
                 break;

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

Gerrit-Project: asterixdb
Gerrit-Branch: ionic
Gerrit-Change-Id: Idee6fb65900b50417f26be88975ef3b186728789
Gerrit-Change-Number: 19488
Gerrit-PatchSet: 8
Gerrit-Owner: Hussain Towaileb <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Hussain Towaileb <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Peeyush Gupta <[email protected]>
Gerrit-MessageType: merged

Reply via email to