[jira] [Commented] (HIVE-16584) Warning messages should use LogHelper.printInfo instead of printing to the infoStream directly

2017-05-12 Thread Peter Vary (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-16584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16007846#comment-16007846 ] Peter Vary commented on HIVE-16584: --- Thanks for the review and the commit [~thejas]! > Warning messages

[jira] [Commented] (HIVE-16584) Warning messages should use LogHelper.printInfo instead of printing to the infoStream directly

2017-05-11 Thread Peter Vary (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-16584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16006116#comment-16006116 ] Peter Vary commented on HIVE-16584: --- Thanks for the review [~thejas]! Could you please commit? I am not

[jira] [Commented] (HIVE-16584) Warning messages should use LogHelper.printInfo instead of printing to the infoStream directly

2017-05-09 Thread Thejas M Nair (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-16584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16003334#comment-16003334 ] Thejas M Nair commented on HIVE-16584: -- +1 Thanks for the great test coverage as well! > Warning

[jira] [Commented] (HIVE-16584) Warning messages should use LogHelper.printInfo instead of printing to the infoStream directly

2017-05-09 Thread Peter Vary (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-16584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16002471#comment-16002471 ] Peter Vary commented on HIVE-16584: --- Could you please review [~thejas], [~ashutoshc]? Thanks, Peter >

[jira] [Commented] (HIVE-16584) Warning messages should use LogHelper.printInfo instead of printing to the infoStream directly

2017-05-08 Thread Peter Vary (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-16584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16000618#comment-16000618 ] Peter Vary commented on HIVE-16584: --- The test failures are not related > Warning messages should use

[jira] [Commented] (HIVE-16584) Warning messages should use LogHelper.printInfo instead of printing to the infoStream directly

2017-05-08 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-16584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16000616#comment-16000616 ] Hive QA commented on HIVE-16584: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-16584) Warning messages should use LogHelper.printInfo instead of printing to the infoStream directly

2017-05-06 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-16584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15999421#comment-15999421 ] Hive QA commented on HIVE-16584: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-16584) Warning messages should use LogHelper.printInfo instead of printing to the infoStream directly

2017-05-06 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-16584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15999365#comment-15999365 ] Hive QA commented on HIVE-16584: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-16584) Warning messages should use LogHelper.printInfo instead of printing to the infoStream directly

2017-05-04 Thread Ashutosh Chauhan (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-16584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15997413#comment-15997413 ] Ashutosh Chauhan commented on HIVE-16584: - since we want that message to be printed on beeline as

[jira] [Commented] (HIVE-16584) Warning messages should use LogHelper.printInfo instead of printing to the infoStream directly

2017-05-04 Thread Thejas M Nair (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-16584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15997401#comment-15997401 ] Thejas M Nair commented on HIVE-16584: -- [~ashutoshc] Thats because its using printInfoNoLog. It

[jira] [Commented] (HIVE-16584) Warning messages should use LogHelper.printInfo instead of printing to the infoStream directly

2017-05-04 Thread Ashutosh Chauhan (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-16584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15997367#comment-15997367 ] Ashutosh Chauhan commented on HIVE-16584: - Missing stats warning is also reported as not being

[jira] [Commented] (HIVE-16584) Warning messages should use LogHelper.printInfo instead of printing to the infoStream directly

2017-05-04 Thread Thejas M Nair (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-16584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15997357#comment-15997357 ] Thejas M Nair commented on HIVE-16584: -- +1 Some warnings are missing as a result. Great minds think