[ 
https://issues.apache.org/jira/browse/HIVE-14111?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15352351#comment-15352351
 ] 

Siddharth Seth commented on HIVE-14111:
---------------------------------------

FileSinkOperator, Operation, MetadataOperation, SQLOperation changes carried 
forward from some other patch ?

The try { } finally {} in TezTask needs to deal with one special condition. An 
attempt to submit() dag can result in session.destroy being invoked - in which 
case I believe the session is lost again, and will never be added back to the 
queue.

Also - while going through this code, I noticed some other interesting bits. 
Will post that as a diff - let me know what you think. Definitely seems like a 
part2 will be required, and unit tests.


> better concurrency handling for TezSessionState - part I
> --------------------------------------------------------
>
>                 Key: HIVE-14111
>                 URL: https://issues.apache.org/jira/browse/HIVE-14111
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Sergey Shelukhin
>            Assignee: Sergey Shelukhin
>         Attachments: HIVE-14111.patch
>
>




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

Reply via email to