----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54305/#review160050 -----------------------------------------------------------
contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/Aggregator.java (line 208) <https://reviews.apache.org/r/54305/#comment231057> can you move .append(")") out of this if block. - Prabhjyot Singh On Dec. 2, 2016, 12:24 p.m., Nitiraj Rathore wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54305/ > ----------------------------------------------------------- > > (Updated Dec. 2, 2016, 12:24 p.m.) > > > Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Pallav > Kulshreshtha, Rohit Choudhary, and Ashwin Rajeev. > > > Bugs: AMBARI-19062 > https://issues.apache.org/jira/browse/AMBARI-19062 > > > Repository: ambari > > > Description > ------- > > Earlier : ATS calls for each job was taking time. > In this patch : remove ATS calls as now the view will store correct status in > the view tables itself > > > Diffs > ----- > > > contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/Aggregator.java > 99faeca > > Diff: https://reviews.apache.org/r/54305/diff/ > > > Testing > ------- > > manual testing done. > > > Thanks, > > Nitiraj Rathore > >
