[GitHub] spark pull request: [SPARK-3286] - Cannot view ApplicationMaster U...

2014-09-10 Thread benoyantony
Github user benoyantony commented on the pull request: https://github.com/apache/spark/pull/2276#issuecomment-55151029 Thank you. Could one of you please add me as Contributor so that I can assign this jira to me ? --- If your project is set up for it, you can reply

[GitHub] spark pull request: [SPARK-3286] - Cannot view ApplicationMaster U...

2014-09-09 Thread benoyantony
Github user benoyantony commented on a diff in the pull request: https://github.com/apache/spark/pull/2276#discussion_r17321277 --- Diff: yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala --- @@ -189,7 +189,11 @@ private[spark] class

[GitHub] spark pull request: [SPARK-3286] - Cannot view ApplicationMaster U...

2014-09-05 Thread benoyantony
Github user benoyantony commented on the pull request: https://github.com/apache/spark/pull/2276#issuecomment-54652578 Sure. I'll do both. Does Alpha corresponds to Hadoop versions before YARN-1203 ? As you know, before YARN-1203, we cannot pass AM URLS with scheme

[GitHub] spark pull request: [SPARK-3286] - Cannot view ApplicationMaster U...

2014-09-05 Thread benoyantony
Github user benoyantony commented on the pull request: https://github.com/apache/spark/pull/2276#issuecomment-54694835 Thanks . I have taken a different approach. I check if the HTTPS is enabled for YARN by looking at yarn.http.policy. This is a new config parameter

[GitHub] spark pull request: [SPARK-3286] - Cannot view ApplicationMaster U...

2014-09-04 Thread benoyantony
GitHub user benoyantony opened a pull request: https://github.com/apache/spark/pull/2276 [SPARK-3286] - Cannot view ApplicationMaster UI when Yarn’s url scheme i... ...s https You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] spark pull request: [SPARK-3286] - Cannot view ApplicationMaster U...

2014-09-04 Thread benoyantony
Github user benoyantony closed the pull request at: https://github.com/apache/spark/pull/2206 --- 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

[GitHub] spark pull request: [SPARK-3286] - Cannot view ApplicationMaster U...

2014-09-04 Thread benoyantony
Github user benoyantony commented on the pull request: https://github.com/apache/spark/pull/2206#issuecomment-54519466 I have created a new Pull request : https://github.com/apache/spark/pull/2276. I have added the code to remove the url scheme for the alpha version. I am

[GitHub] spark pull request: [SPARK-3286] - Cannot view ApplicationMaster U...

2014-09-04 Thread benoyantony
Github user benoyantony commented on a diff in the pull request: https://github.com/apache/spark/pull/2276#discussion_r17131360 --- Diff: yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala --- @@ -188,7 +188,7 @@ private[spark] class ApplicationMaster

[GitHub] spark pull request: [SPARK-3287] [YARN]When ResourceManager High A...

2014-09-04 Thread benoyantony
Github user benoyantony commented on the pull request: https://github.com/apache/spark/pull/2212#issuecomment-54528156 Sure. I can create a new patch. If I cannot depend on WebAppUtils, what's the preferred alternate approach ? Should I duplicate the logic in SPARK

[GitHub] spark pull request: SPARK-3286 - Cannot view ApplicationMaster UI ...

2014-08-29 Thread benoyantony
GitHub user benoyantony opened a pull request: https://github.com/apache/spark/pull/2206 SPARK-3286 - Cannot view ApplicationMaster UI when Yarn’s url scheme is ... ...https You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] spark pull request: [SPARK-3286] - Cannot view ApplicationMaster U...

2014-08-29 Thread benoyantony
Github user benoyantony commented on a diff in the pull request: https://github.com/apache/spark/pull/2206#discussion_r16924081 --- Diff: yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala --- @@ -174,7 +174,7 @@ private[spark] class ApplicationMaster

[GitHub] spark pull request: [SPARK-3287] When ResourceManager High Availab...

2014-08-29 Thread benoyantony
GitHub user benoyantony opened a pull request: https://github.com/apache/spark/pull/2212 [SPARK-3287] When ResourceManager High Availability is enabled, Applicat... ...ionMaster webUI is not displayed You can merge this pull request into a Git repository by running: $ git pull

[GitHub] spark pull request: [SPARK-3286] - Cannot view ApplicationMaster U...

2014-08-29 Thread benoyantony
Github user benoyantony commented on a diff in the pull request: https://github.com/apache/spark/pull/2206#discussion_r16925510 --- Diff: yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala --- @@ -174,7 +174,7 @@ private[spark] class ApplicationMaster

[GitHub] spark pull request: [SPARK-3286] - Cannot view ApplicationMaster U...

2014-08-29 Thread benoyantony
Github user benoyantony commented on a diff in the pull request: https://github.com/apache/spark/pull/2206#discussion_r16928155 --- Diff: yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala --- @@ -174,7 +174,7 @@ private[spark] class ApplicationMaster

[GitHub] spark pull request: [SPARK-3287] When ResourceManager High Availab...

2014-08-29 Thread benoyantony
Github user benoyantony commented on the pull request: https://github.com/apache/spark/pull/2212#issuecomment-53949891 For older versions (versions older than 2.4) , I need to pass the proxyhost information as before. So I can add a check for that. --- If your project is set up