You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@ambari.apache.org by Prabhjyot Singh <pr...@gmail.com> on 2016/12/16 11:08:38 UTC
Review Request 54810: %livy interpreter gives 400 HttpClientError due
to spark.master property present in livy-spark-blacklist
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54810/
-----------------------------------------------------------
Review request for Ambari, Alejandro Fernandez, DIPAYAN BHOWMICK, Gaurav Nagar, Jayush Luniya, Nitiraj Rathore, Rohit Choudhary, Renjith Kamath, and Sumit Mohanty.
Bugs: AMBARI-19222
https://issues.apache.org/jira/browse/AMBARI-19222
Repository: ambari
Description
-------
spark.master should not be overridden because we want to use livy's own default = cluster not overridden by the user via zeppelin. so it should be present in the blacklist if I understand right. zeppelin should stop specifying livy.spark.master in the livy interpreter in case it does so now.
Diffs
-----
ambari-server/src/main/resources/common-services/ZEPPELIN/0.6.0.2.5/package/scripts/master.py 9cc808b
Diff: https://reviews.apache.org/r/54810/diff/
Testing
-------
Manually tested on centos6.6
Thanks,
Prabhjyot Singh
Re: Review Request 54810: %livy interpreter gives 400 HttpClientError
due to spark.master property present in livy-spark-blacklist
Posted by Renjith Kamath <re...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54810/#review159431
-----------------------------------------------------------
Ship it!
Ship It!
- Renjith Kamath
On Dec. 16, 2016, 11:08 a.m., Prabhjyot Singh wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54810/
> -----------------------------------------------------------
>
> (Updated Dec. 16, 2016, 11:08 a.m.)
>
>
> Review request for Ambari, Alejandro Fernandez, DIPAYAN BHOWMICK, Gaurav Nagar, Jayush Luniya, Nitiraj Rathore, Rohit Choudhary, Renjith Kamath, and Sumit Mohanty.
>
>
> Bugs: AMBARI-19222
> https://issues.apache.org/jira/browse/AMBARI-19222
>
>
> Repository: ambari
>
>
> Description
> -------
>
> spark.master should not be overridden because we want to use livy's own default = cluster not overridden by the user via zeppelin. so it should be present in the blacklist if I understand right. zeppelin should stop specifying livy.spark.master in the livy interpreter in case it does so now.
>
>
> Diffs
> -----
>
> ambari-server/src/main/resources/common-services/ZEPPELIN/0.6.0.2.5/package/scripts/master.py 9cc808b
>
> Diff: https://reviews.apache.org/r/54810/diff/
>
>
> Testing
> -------
>
> Manually tested on centos6.6
>
>
> Thanks,
>
> Prabhjyot Singh
>
>
Re: Review Request 54810: %livy interpreter gives 400 HttpClientError
due to spark.master property present in livy-spark-blacklist
Posted by Alejandro Fernandez <af...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54810/#review159477
-----------------------------------------------------------
Ship it!
Ship It!
- Alejandro Fernandez
On Dec. 16, 2016, 11:08 a.m., Prabhjyot Singh wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54810/
> -----------------------------------------------------------
>
> (Updated Dec. 16, 2016, 11:08 a.m.)
>
>
> Review request for Ambari, Alejandro Fernandez, DIPAYAN BHOWMICK, Gaurav Nagar, Jayush Luniya, Nitiraj Rathore, Rohit Choudhary, Renjith Kamath, and Sumit Mohanty.
>
>
> Bugs: AMBARI-19222
> https://issues.apache.org/jira/browse/AMBARI-19222
>
>
> Repository: ambari
>
>
> Description
> -------
>
> spark.master should not be overridden because we want to use livy's own default = cluster not overridden by the user via zeppelin. so it should be present in the blacklist if I understand right. zeppelin should stop specifying livy.spark.master in the livy interpreter in case it does so now.
>
>
> Diffs
> -----
>
> ambari-server/src/main/resources/common-services/ZEPPELIN/0.6.0.2.5/package/scripts/master.py 9cc808b
>
> Diff: https://reviews.apache.org/r/54810/diff/
>
>
> Testing
> -------
>
> Manually tested on centos6.6
>
>
> Thanks,
>
> Prabhjyot Singh
>
>