----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65686/#review197767 -----------------------------------------------------------
hcatalog/webhcat/svr/src/main/java/org/apache/hive/hcatalog/templeton/AppConfig.java Lines 113-117 (patched) <https://reviews.apache.org/r/65686/#comment278047> These configs look fine to me. After the patch is committed they will need to be documented in https://cwiki.apache.org/confluence/display/Hive/WebHCat+Configure#WebHCatConfigure-ConfigurationVariables. HIVE-18728's description has the documentation. - Lefty Leverenz On Feb. 16, 2018, 2:19 p.m., Oleksiy Sayankin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65686/ > ----------------------------------------------------------- > > (Updated Feb. 16, 2018, 2:19 p.m.) > > > Review request for hive, Lefty Leverenz, Lefty Leverenz, Rui Li, Lenni Kuff, > Szehon Ho, and Szehon Ho. > > > Repository: hive-git > > > Description > ------- > > Initiall commit > > > Diffs > ----- > > > hcatalog/webhcat/svr/src/main/java/org/apache/hive/hcatalog/templeton/AppConfig.java > 1fd9e47 > > hcatalog/webhcat/svr/src/main/java/org/apache/hive/hcatalog/templeton/Main.java > d183b2e > > > Diff: https://reviews.apache.org/r/65686/diff/2/ > > > Testing > ------- > > > Thanks, > > Oleksiy Sayankin > >