----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47177/#review132889 -----------------------------------------------------------
contrib/views/hive/src/main/resources/ui/hive-web/app/controllers/upload-table.js (line 90) <https://reviews.apache.org/r/47177/#comment197170> Remove console staemnets. contrib/views/hive/src/main/resources/ui/hive-web/app/controllers/upload-table.js (line 236) <https://reviews.apache.org/r/47177/#comment197169> Use map and don't introduce extra variable 'defaultColumnNames' - Pallav Kulshreshtha On May 10, 2016, 10:49 a.m., Nitiraj Rathore wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47177/ > ----------------------------------------------------------- > > (Updated May 10, 2016, 10:49 a.m.) > > > Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Pallav > Kulshreshtha, and Ashwin Rajeev. > > > Bugs: AMBARI-16424 > https://issues.apache.org/jira/browse/AMBARI-16424 > > > Repository: ambari > > > Description > ------- > > Earlier : the header column names used to remain the same as the first row of > the file. > > In this patch : UI has been changed to show the default column names > "column1" etc. > > > Diffs > ----- > > > contrib/views/hive/src/main/resources/ui/hive-web/app/controllers/upload-table.js > d883743 > > Diff: https://reviews.apache.org/r/47177/diff/ > > > Testing > ------- > > Manual testing done. > > > Thanks, > > Nitiraj Rathore > >