----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54872/#review159651 -----------------------------------------------------------
Ship it! Ship It! - Tim Thorpe On Dec. 19, 2016, 7:52 p.m., Di Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54872/ > ----------------------------------------------------------- > > (Updated Dec. 19, 2016, 7:52 p.m.) > > > Review request for Ambari and Tim Thorpe. > > > Bugs: AMBARI-19226 > https://issues.apache.org/jira/browse/AMBARI-19226 > > > Repository: ambari > > > Description > ------- > > provide a default value in params.py for hdfs_tmp_dir. The property is not > shown on the UI, making it difficult for users to change its value via Ambari > web UI if the cluster is deployed via blueprint and the property is missing. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/params_linux.py > 7567ab4 > > ambari-server/src/main/resources/common-services/HDFS/3.0.0.3.0/package/scripts/params_linux.py > 55544e0 > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/params_linux.py > bebb375 > > ambari-server/src/main/resources/common-services/YARN/3.0.0.3.0/package/scripts/params_linux.py > cdadc80 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/params.py > daba3d3 > > ambari-server/src/main/resources/stacks/HDP/3.0/hooks/before-START/scripts/params.py > d838211 > > Diff: https://reviews.apache.org/r/54872/diff/ > > > Testing > ------- > > exsiting unit tests > Patch a 2.4 cluster (Ambari upgraded to trunk build) with the changes, run > EU, then run downgrade once it reaches the Finalize step. > > > Thanks, > > Di Li > >