----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28014/#review61341 -----------------------------------------------------------
Ship it! Ship It! - Jaimin Jetly On Nov. 14, 2014, 12:16 a.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28014/ > ----------------------------------------------------------- > > (Updated Nov. 14, 2014, 12:16 a.m.) > > > Review request for Ambari, Jaimin Jetly, Mahadev Konar, Sumit Mohanty, and > Sid Wagle. > > > Bugs: AMBARI-8323 > https://issues.apache.org/jira/browse/AMBARI-8323 > > > Repository: ambari > > > Description > ------- > > The WebHcat server tries to copy the pig.tar.gz file to HDFS if the file > exists in /usr/hdp/current/pig-client/pig.tar.gz. > The file will only exist there if the host also has the Pig client installed. > For this reason, installing the WebHCat server must also install the Pig > client, and this should work for both the Install Wizard and Blueprints. > > > Diffs > ----- > > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/metainfo.xml > cf2768e > > Diff: https://reviews.apache.org/r/28014/diff/ > > > Testing > ------- > > Did end-to-end test on a 3-node cluster where 1 host was the master and > contained all servers and no clients, and the other 2 hosts only contained > clients. The primary host that also had WebHCat server did end up installing > Pig client and copying pig.tar.gz to HDFS as expected. > I also verified this through Blueprints. > > mvn clean test > > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 27:04.012s > [INFO] Finished at: Thu Nov 13 16:09:17 PST 2014 > [INFO] Final Memory: 61M/623M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Alejandro Fernandez > >
