Skip to content

[SPARK-32034][SQL][2.4] Port HIVE-14817: Shutdown the SessionManager timeoutChecker thread properly upon shutdown - #28888

Closed
yaooqinn wants to merge 1 commit into
apache:branch-2.4from
yaooqinn:SPARK-32034-24
Closed

[SPARK-32034][SQL][2.4] Port HIVE-14817: Shutdown the SessionManager timeoutChecker thread properly upon shutdown#28888
yaooqinn wants to merge 1 commit into
apache:branch-2.4from
yaooqinn:SPARK-32034-24

Conversation

@yaooqinn

@yaooqinnyaooqinn commented Jun 22, 2020

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

This PR backports #28870 which ports https://issues.apache.org/jira/browse/HIVE-14817 for spark thrift server.

Why are the changes needed?

Port HIVE-14817 to fix related issues

Does this PR introduce any user-facing change?

no

How was this patch tested?

passing Jenkins

@yaooqinn

Copy link
Copy Markdown
MemberAuthor

cc @dongjoon-hyun, thanks for reviewing this pr in advance

@SparkQA

Copy link
Copy Markdown

Test build #124346 has finished for PR 28888 at commit 23f3dda.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@dongjoon-hyundongjoon-hyun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM. Thank you, @yaooqinn .
Merged to branch-2.4.

dongjoon-hyun pushed a commit that referenced this pull request Jun 22, 2020
…timeoutChecker thread properly upon shutdown
### What changes were proposed in this pull request?
This PR backports #28870 which ports https://issues.apache.org/jira/browse/HIVE-14817 for spark thrift server.
### Why are the changes needed?
Port HIVE-14817 to fix related issues
### Does this PR introduce _any_ user-facing change?
no
### How was this patch tested?
passing Jenkins
Closes#28888 from yaooqinn/SPARK-32034-24.
Authored-by: Kent Yao <yaooqinn@hotmail.com>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@yaooqinn@SparkQA@dongjoon-hyun