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

Brock Noland commented on HIVE-5799:
------------------------------------

# It appears the constant SessionManager. SESSION_CHECK_INTERVAL is unused.
# HiveSessionImpl. opHandleSet should be converted to a synchronized set or a 
concurrent set since it's now modified by the client thread and the background 
thread
# I think we should call OperationManager._getOperation  getOperationInternal 
as that seems to be more standard for this use case in the Hive code base.
# Perhaps we should change OperationManager.closeExpiredOperations to not use 
closeOperation but use similar code since it appears there is a harmless race 
condition there which will log exceptions when the operation is closed during 
closeExpiredOperations

Does anyone else have any feedback? Otherwise once these are fixed I think we 
can commit.

> session/operation timeout for hiveserver2
> -----------------------------------------
>
>                 Key: HIVE-5799
>                 URL: https://issues.apache.org/jira/browse/HIVE-5799
>             Project: Hive
>          Issue Type: Improvement
>          Components: HiveServer2
>            Reporter: Navis
>            Assignee: Navis
>            Priority: Minor
>         Attachments: HIVE-5799.1.patch.txt, HIVE-5799.10.patch.txt, 
> HIVE-5799.11.patch.txt, HIVE-5799.12.patch.txt, HIVE-5799.12.patch.txt, 
> HIVE-5799.2.patch.txt, HIVE-5799.3.patch.txt, HIVE-5799.4.patch.txt, 
> HIVE-5799.5.patch.txt, HIVE-5799.6.patch.txt, HIVE-5799.7.patch.txt, 
> HIVE-5799.8.patch.txt, HIVE-5799.9.patch.txt
>
>
> Need some timeout facility for preventing resource leakages from instable  or 
> bad clients.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to