HIVE-20043: HiveServer2: SessionState has a static sync block around an AtomicBoolean (Laszlo Bodor, reviewed by Zoltan Haindrich)
Signed-off-by: Gopal V <[email protected]> Project: http://git-wip-us.apache.org/repos/asf/hive/repo Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/db1a70eb Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/db1a70eb Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/db1a70eb Branch: refs/heads/master Commit: db1a70eb361e56f57bd10fcc0cbf21b99866fcdf Parents: eb91fa4 Author: Laszlo Bodor <[email protected]> Authored: Mon Jul 9 09:31:43 2018 -0700 Committer: Gopal V <[email protected]> Committed: Mon Jul 9 09:32:18 2018 -0700 ---------------------------------------------------------------------- .../java/org/apache/hadoop/hive/ql/session/SessionState.java | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hive/blob/db1a70eb/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java ---------------------------------------------------------------------- diff --git a/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java b/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java index 27f0216..262bbb9 100644 --- a/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java +++ b/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java @@ -607,10 +607,8 @@ public class SessionState { private static void start(SessionState startSs, boolean isAsync, LogHelper console) { setCurrentSessionState(startSs); - synchronized(SessionState.class) { - if (!startSs.isStarted.compareAndSet(false, true)) { - return; - } + if (!startSs.isStarted.compareAndSet(false, true)) { + return; } if (startSs.hiveHist == null){
