> On March 13, 2017, 6:32 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/SPARK2/2.0.0/package/scripts/spark_service.py > > Line 66 (original), 66 (patched) > > <https://reviews.apache.org/r/57557/diff/1/?file=1662688#file1662688line66> > > > > Why does this need to replace the existing tarball? > > The tarball comes from a specific version which shouldn't be changing? > > I'm concerned this will cause the Spark Service Check to be slower.
This code does not run during service check, only when Spark2 History Server is started. - Attila ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57557/#review168789 ----------------------------------------------------------- On March 13, 2017, 1:47 p.m., Attila Doroszlai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57557/ > ----------------------------------------------------------- > > (Updated March 13, 2017, 1:47 p.m.) > > > Review request for Ambari, Bikas Saha, Laszlo Puskas, and Sandor Magyari. > > > Bugs: AMBARI-20417 > https://issues.apache.org/jira/browse/AMBARI-20417 > > > Repository: ambari > > > Description > ------- > > Fix directory permissions even if already present. Force upload to HDFS. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/SPARK2/2.0.0/package/scripts/spark_service.py > c8a2b33ff158fe781926bf88c03be471fa9de1ca > > > Diff: https://reviews.apache.org/r/57557/diff/1/ > > > Testing > ------- > > Manual test according to steps to reproduce. > > > Thanks, > > Attila Doroszlai > >
