----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60480/#review179142 -----------------------------------------------------------
ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/hive-env.xml Lines 119 (patched) <https://reviews.apache.org/r/60480/#comment253664> This will cause upgrades to change configs and have Hive report staleness and require a restart. Instead, it seems like we should default to False if not found and use the upgrade packs to update this. In other words; since Stack-A might need this when moving to Stack-B, then it should provide this as a configuration setting. - Jonathan Hurley On June 28, 2017, 1:31 p.m., Di Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60480/ > ----------------------------------------------------------- > > (Updated June 28, 2017, 1:31 p.m.) > > > Review request for Ambari, Dmytro Grinenko, Di Li, Dmitro Lisnichenko, > Jonathan Hurley, Sumit Mohanty, Sid Wagle, and Tim Thorpe. > > > Bugs: AMBARI-21351 > https://issues.apache.org/jira/browse/AMBARI-21351 > > > Repository: ambari > > > Description > ------- > > Some stacks, such as BigInights, support MariaDB as the database for Hive. > When upgrading from BigInights to another stack, such as HDP, this causes the > following exception when starting the Hive service: > ```resource_management.core.exceptions.ExecutionFailed: Execution of 'service > mysqld stop' returned 5. Redirecting to /bin/systemctl stop mysqld.service > Failed to stop mysqld.service: Unit mysqld.service not loaded.``` > This looks to be caused by the hard-coded reliance of mysqld in the Python > scripts. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/hive-env.xml > 1cf9016 > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/status_params.py > b7cb148 > > > Diff: https://reviews.apache.org/r/60480/diff/2/ > > > Testing > ------- > > Tim and I patched a cluster with basically the same fix when migrating an IOP > 4.2.5 cluster to HDP 2.6/Ambari 2.5.1. THis patch has one additional change > that's to set StackFeature boolean in case this code gets merged back to > trunk later on. > > If the code only stays in this branch, the StackFeature boolean change can be > omitted. > > > Thanks, > > Di Li > >