----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50510/#review143886 -----------------------------------------------------------
beeline/src/java/org/apache/hive/beeline/BeeLineOpts.java (line 627) <https://reviews.apache.org/r/50510/#comment209837> Adding @Ignore may not be the right solution here. This will skip saving the conf variable which can potentially lead to loss of user's custom settings for HiveConf. The issue is caused because the method name is setHiveConf while the Reflector is looking for setConf. Did you try renaming the method from setHiveConf to setConf? - Vihang Karajgaonkar On July 27, 2016, 4:26 p.m., Peter Vary wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50510/ > ----------------------------------------------------------- > > (Updated July 27, 2016, 4:26 p.m.) > > > Review request for hive, Aihua Xu, Sergio Pena, and Vihang Karajgaonkar. > > > Bugs: HIVE-14360 > https://issues.apache.org/jira/browse/HIVE-14360 > > > Repository: hive-git > > > Description > ------- > > Adding the @Ignore annotation to the getConf method, so the saving should > ignore the attribute > > > Diffs > ----- > > beeline/src/java/org/apache/hive/beeline/BeeLineOpts.java e6e24b1 > beeline/src/test/org/apache/hive/beeline/TestBeeLineOpts.java PRE-CREATION > > Diff: https://reviews.apache.org/r/50510/diff/ > > > Testing > ------- > > Manually + unit test > > > Thanks, > > Peter Vary > >
