[jira] [Updated] (HIVE-10441) Fix confusing log statement in SessionState about hive.execution.engine setting

2015-04-27 Thread Jason Dere (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-10441?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jason Dere updated HIVE-10441:
--
Fix Version/s: (was: 1.3.0)
   1.2.0

 Fix confusing log statement in SessionState about hive.execution.engine 
 setting
 ---

 Key: HIVE-10441
 URL: https://issues.apache.org/jira/browse/HIVE-10441
 Project: Hive
  Issue Type: Bug
Reporter: Jason Dere
Assignee: Jason Dere
 Fix For: 1.2.0

 Attachments: HIVE-10441.1.patch


 {code}
 LOG.info(No Tez session required at this point. hive.execution.engine=mr.);
 {code}
 This statement is misleading. It is true that it is printed in the case that 
 Tez session does not need to be created, but it is not necessarily true that 
 hive.execution.engine=mr - it could be Spark, or it could even be set to Tez 
 but the Session determined that a Tez Session did not need to be created 
 (which is the case for HiveServer2).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HIVE-10441) Fix confusing log statement in SessionState about hive.execution.engine setting

2015-04-22 Thread Jason Dere (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-10441?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jason Dere updated HIVE-10441:
--
Attachment: HIVE-10441.1.patch

I don't see a lot of value in having this statement here to mention that no Tez 
Session is necessary, because that is redundant for mr/spark. Also, if a Tez 
session is created, there are log statements elsewhere for that. I'm just going 
to remove this log statement.

 Fix confusing log statement in SessionState about hive.execution.engine 
 setting
 ---

 Key: HIVE-10441
 URL: https://issues.apache.org/jira/browse/HIVE-10441
 Project: Hive
  Issue Type: Bug
Reporter: Jason Dere
Assignee: Jason Dere
 Attachments: HIVE-10441.1.patch


 {code}
 LOG.info(No Tez session required at this point. hive.execution.engine=mr.);
 {code}
 This statement is misleading. It is true that it is printed in the case that 
 Tez session does not need to be created, but it is not necessarily true that 
 hive.execution.engine=mr - it could be Spark, or it could even be set to Tez 
 but the Session determined that a Tez Session did not need to be created 
 (which is the case for HiveServer2).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)