[GitHub] [hive] zhangbutao commented on pull request #3542: HIVE-16913: Support per-session S3 credentials

[GitBox <gi...@apache.org>]

zhangbutao commented on PR #3542:
URL: https://github.com/apache/hive/pull/3542#issuecomment-1231201583

   > Can we a have a test (either unit test or qtest) for this? with a dummy keys and printing it somewhere
   
   @shameersss1 Thx for your advice. This PR seems difficult to test using current test framework. Although HIVE-14373 has added integration tests for hive on S3, it is mainly tested using global ak & sk. 
   However,  this PR is aimed for dynamic ak & sk, and I need set the ak & sk parameters using hive set command. Maybe i need sometime to explore how to test this scenario...


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org