[
https://issues.apache.org/jira/browse/HIVE-22165?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16922252#comment-16922252
]
Amruth S edited comment on HIVE-22165 at 9/4/19 7:53 AM:
---------------------------------------------------------
Oops sorry. I had incorrectly created the diff earlier. Thanks for pointing out.
was (Author: amrk7):
Oops sorry. I had incorrectly created the diff
> Synchronisation introduced by HIVE-14296 on SessionManager.closeSession
> causes high latency in a busy hive server
> -----------------------------------------------------------------------------------------------------------------
>
> Key: HIVE-22165
> URL: https://issues.apache.org/jira/browse/HIVE-22165
> Project: Hive
> Issue Type: Bug
> Components: HiveServer2
> Affects Versions: 2.1.0, 2.3.2
> Reporter: Amruth S
> Assignee: Amruth S
> Priority: Major
> Attachments: HIVE-22165.patch, HIVE-22165.patch.1
>
>
> HIVE-14296 introduces this
> [commit|https://github.com/apache/hive/commit/477a47d3b4b9e3da3c22465217c2024588f7f000]
> which adds synchronization to SessionManager.closeSession.
> And it looks like it is used only for logging purposes.
> In a busy hive server where 5-10 sessions are created closed every second, an
> increase in latency of any other downstream services (Zk, HDFS) causes a
> queueing effect (lot of threads getting blocked on
> SessionManager.closeSession) creating an induced latency of 3-5 minutes at
> times for just closing the session.
> Since the gauge (MetricsConstant.HS2_OPEN_SESSIONS) is already tracking the
> open session counts, the synchronization (along with the additional logging)
> can be without any functionality losses.
--
This message was sent by Atlassian Jira
(v8.3.2#803003)
