> On June 4, 2015, 2:50 a.m., Jonathan Hurley wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/copy_tarball.py, > > line 64 > > <https://reviews.apache.org/r/34982/diff/2/?file=978063#file978063line64> > > > > I think we need to use the temp directory defined in the agent's config > > for any temporary files. > > Nate Cole wrote: > I had suggested this once - the Script and r_m library don't have access > to the agent config. That being said, the temp directory IS passed in to a > script, so we should probably pass it along and use it here too. > > Andrew Onischuk wrote: > actually you can just Script.get_tmp_dir() for that > > Nate Cole wrote: > @Andrew - good one!
thanks) - Andrew ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34982/#review86540 ----------------------------------------------------------- On June 4, 2015, 12:23 a.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34982/ > ----------------------------------------------------------- > > (Updated June 4, 2015, 12:23 a.m.) > > > Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, Jonathan > Hurley, Nate Cole, Srimanth Gunturi, and Sid Wagle. > > > Bugs: AMBARI-11640 > https://issues.apache.org/jira/browse/AMBARI-11640 > > > Repository: ambari > > > Description > ------- > > The workaround is to start all services so that the CURRENT version exists, > and then rerun a HiveServer2 Start, Spark History Server2, or Tez Service > Check. > The fix will be to detect if params.current_version is None, and if so, call > hdp-select versions, and use the value if only one version number is reported. > Also, HiveServer2 Start needs to copy the tez tarball. > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/copy_tarball.py > f8a2570 > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/atlas_plugin_utils.py > d057c27 > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive.py > 32edf71 > ambari-server/src/test/python/stacks/2.0.6/HIVE/test_hive_server.py 051b8c3 > > Diff: https://reviews.apache.org/r/34982/diff/ > > > Testing > ------- > > Verified this worked on a brand new cluster install with HistoryServer Start, > which copies mapreduce.tar.gz to HDFS. > > ---------------------------------------------------------------------- > Total run:747 > Total errors:0 > Total failures:0 > OK > > > Thanks, > > Alejandro Fernandez > >
