----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54902/#review159725 -----------------------------------------------------------
Ship it! Ship It! - Tim Thorpe On Dec. 20, 2016, 4:10 p.m., Di Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54902/ > ----------------------------------------------------------- > > (Updated Dec. 20, 2016, 4:10 p.m.) > > > Review request for Ambari and Tim Thorpe. > > > Bugs: AMBARI-19247 > https://issues.apache.org/jira/browse/AMBARI-19247 > > > Repository: ambari > > > Description > ------- > > there is currently no log indicating mismatched roles and the corresponding > behavior on skipping checking role (component) versions when the service > status map or service check map does not have the role. > > there should be minimal logging to be included in output-[taskid].out > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/stack_select.py > 513ceac > > Diff: https://reviews.apache.org/r/54902/diff/ > > > Testing > ------- > > patched a trunk cluster with the change and run thru express upgrade. > > > Thanks, > > Di Li > >