[
https://issues.apache.org/jira/browse/HIVE-15563?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15819748#comment-15819748
]
zhihai xu commented on HIVE-15563:
--
Thanks [~jxiang] for reviewing and committing the patch!
> Ignore
[
https://issues.apache.org/jira/browse/HIVE-15563?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15817004#comment-15817004
]
Hive QA commented on HIVE-15563:
Here are the results of testing the latest attachment:
[
https://issues.apache.org/jira/browse/HIVE-15563?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15816399#comment-15816399
]
Jimmy Xiang commented on HIVE-15563:
Thanks for making the change. Looks good to me. +1
> Ignore
[
https://issues.apache.org/jira/browse/HIVE-15563?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15816369#comment-15816369
]
zhihai xu commented on HIVE-15563:
--
thanks for the review [~jxiang]! Good suggestions, In this case,
[
https://issues.apache.org/jira/browse/HIVE-15563?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15815852#comment-15815852
]
Jimmy Xiang commented on HIVE-15563:
Instead of catching the state transition exception, should we
[
https://issues.apache.org/jira/browse/HIVE-15563?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15810197#comment-15810197
]
Hive QA commented on HIVE-15563:
Here are the results of testing the latest attachment: