Re: Review Request 47579: Spark2 service definition for Ambari

2016-06-02 Thread Jayush Luniya
> On May 25, 2016, 8:29 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/SPARK2/2.0.0/package/scripts/setup_spark.py, > > line 25 > > > > > > Use explicit import > > Jayush

Re: Review Request 47579: Spark2 service definition for Ambari

2016-05-26 Thread Jayush Luniya
> On May 25, 2016, 8:29 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/SPARK2/2.0.0/package/scripts/setup_spark.py, > > line 25 > > > > > > Use explicit import Sorry I missed

Re: Review Request 47579: Spark2 service definition for Ambari

2016-05-25 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47579/#review134827 --- Fix it, then Ship it!

Re: Review Request 47579: Spark2 service definition for Ambari

2016-05-22 Thread Saisai Shao
> On May 22, 2016, 1:28 p.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/SPARK2/2.0.0/configuration/spark2-env.xml, > > line 71 > > > > > > spark2-env Same as above. - Saisai

Re: Review Request 47579: Spark2 service definition for Ambari

2016-05-22 Thread Saisai Shao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47579/ --- (Updated May 23, 2016, 2:55 a.m.) Review request for Ambari, Jayush Luniya and

Re: Review Request 47579: Spark2 service definition for Ambari

2016-05-22 Thread Saisai Shao
> On May 22, 2016, 1:28 p.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/SPARK2/2.0.0/configuration/spark2-env.xml, > > line 63 > > > > > > spark2-env Actually we don't change the

Re: Review Request 47579: Spark2 service definition for Ambari

2016-05-22 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47579/#review134269 ---