> On Dec. 5, 2017, 4:56 p.m., Jonathan Hurley wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/lzo_utils.py > > Lines 72 (patched) > > <https://reviews.apache.org/r/64315/diff/2/?file=1909085#file1909085line75> > > > > Maybe also logger.info "This stack does not indicate that it supports > > LZO installation"
Good point. Thanks for the quick review will make those changes before commit. - Sid ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64315/#review192882 ----------------------------------------------------------- On Dec. 5, 2017, 4:51 p.m., Sid Wagle wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64315/ > ----------------------------------------------------------- > > (Updated Dec. 5, 2017, 4:51 p.m.) > > > Review request for Ambari, Aravindan Vijayan and Jonathan Hurley. > > > Bugs: AMBARI-22588 > https://issues.apache.org/jira/browse/AMBARI-22588 > > > Repository: ambari > > > Description > ------- > > IOP Migration: Hive Client restart fails: hadooplzo package not present in > iop repos > > *STR*: > - Install IOP-425 cluster > - Now upgrade Ambari to 2.6.1. Hive client restart fails > > {code} > Cannot match package for regexp name hadooplzo_${stack_version} > {code} > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/lzo_utils.py > 68ee607 > > ambari-server/src/main/resources/stacks/BigInsights/4.0/properties/stack_features.json > d29efb2 > > > Diff: https://reviews.apache.org/r/64315/diff/2/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Sid Wagle > >
