[GitHub] storm pull request: STORM-997: Add proxy user functionality for st...

2015-10-06 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/692#issuecomment-145868091 @priyank5485 You are correct that the name-node needs to be configured to explicitly allow super users and/or groups and optionally restrict those to specific hosts. My

[GitHub] storm pull request: STORM-997: Add proxy user functionality for st...

2015-10-05 Thread priyank5485
Github user priyank5485 commented on the pull request: https://github.com/apache/storm/pull/692#issuecomment-145642316 @revans2 I think what you mentioned above would not happen. I mean the part where you say that storm running in unsecured mode and hdfs in secured mode will disable s

[GitHub] storm pull request: STORM-997: Add proxy user functionality for st...

2015-10-03 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/692#issuecomment-145260515 @priyank5485 and others I am not totally sure that this is a good idea. If HDFS is running secure and Storm is running insecure, with this feature enabled you have now d

[GitHub] storm pull request: STORM-997: Add proxy user functionality for st...

2015-09-24 Thread priyank5485
Github user priyank5485 commented on the pull request: https://github.com/apache/storm/pull/692#issuecomment-143101710 @Parth-Brahmbhatt @ptgoetz Can you please review this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] storm pull request: STORM-997: Add proxy user functionality for st...

2015-08-31 Thread priyank5485
Github user priyank5485 commented on the pull request: https://github.com/apache/storm/pull/692#issuecomment-136490586 @harshach thanks for the review. Yes, I will do it. I realized that it needs to be added for trident after creating the pull request. --- If your project is set up f

[GitHub] storm pull request: STORM-997: Add proxy user functionality for st...

2015-08-30 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/692#issuecomment-136152260 @priyank5485 we need to add this trident as well. --- 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

[GitHub] storm pull request: STORM-997: Add proxy user functionality for st...

2015-08-20 Thread priyank5485
GitHub user priyank5485 opened a pull request: https://github.com/apache/storm/pull/692 STORM-997: Add proxy user functionality for storm hdfs connector. You can merge this pull request into a Git repository by running: $ git pull https://github.com/priyank5485/storm STORM-997