Github user Leemoonsoo commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/860#issuecomment-215940375
  
    Hi @prasadwagle. Thanks for sharing great work.
    
    I think this is nice solution for 
http://apache-zeppelin-users-incubating-mailing-list.75479.x6.nabble.com/Storing-Interpreter-Passwords-tp2773.html
    
    One feedback is, about passing credential information to interpreter 
process.
    Currently, single credential passed to interpreter process, when 
DATSOURCE_KEY matches to '%hive(DATASOURCE_KEY)'.
    
    But currently, hive (jdbc) is only interpreter that leverages 
'%hive(DATASOURCE_KEY)'. So most of other interpreters are not able to leverage 
this new features.
    
    So, how about passing all credential map (of login user) to interpreter 
process?
    And let interpreter find appropriate credential from the map. 
    What do you think?



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to