[ 
https://issues.apache.org/jira/browse/TWILL-189?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15467992#comment-15467992
 ] 

Terence Yim commented on TWILL-189:
-----------------------------------

Shall we remove the {{<T> T getStore()}} method then? Since now's it's the 
{{SecureStore}} responsibility to write to a given location? By the way, 
instead of giving a {{URI}}, I think it's better to have the method as:

{{void persist(OutputProvider<OutputStream> osProvider)}}

with {{OutputProvider}} has a single method {{OutputStream open()}}.

With that, the user doesn't need to handle the URI by himself. With this API, I 
believe we also need to fix the {{LocationFactory}} implementations (HDFS and 
FileContext) to have the {{create}} method always use the current UGI to open 
the stream, instead of the cached one.

> SecureStoreUpdater should support updating secure store as other users
> ----------------------------------------------------------------------
>
>                 Key: TWILL-189
>                 URL: https://issues.apache.org/jira/browse/TWILL-189
>             Project: Apache Twill
>          Issue Type: New Feature
>          Components: api
>    Affects Versions: 0.7.0-incubating
>            Reporter: Ali Anwar
>             Fix For: 0.8.0
>
>
> SecureStoreUpdater always creates secure stores for the current user and 
> writes these as the current user.
> It should allow writing the secure store as a different user.
> For an example use case, see: https://github.com/caskdata/cdap/pull/6380



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to