Skip to content

Conversation

@cloud-fan
Copy link
Contributor

This reverts commit b83b502.

It has a regression as reported in #19068 (comment) , seems the fix is not trivial, we should revert this commit first.

This revert has some conflicts, so try to go through jenkins to avoid to mess up something.

…in Hive jars for reusing CliSessionState"

This reverts commit b83b502.
@cloud-fan
Copy link
Contributor Author

cc @yaooqinn @jiangxb1987

@yaooqinn
Copy link
Member

ok to me, more tests are needed on #18648

@SparkQA
Copy link

SparkQA commented Sep 19, 2017

Test build #81908 has finished for PR 19273 at commit 72c5a8e.

  • This patch fails due to an unknown error code, -9.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Sep 19, 2017

Test build #81915 has finished for PR 19273 at commit 72c5a8e.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@cloud-fan
Copy link
Contributor Author

closing it as the regression is fixed by #19068

@cloud-fan cloud-fan closed this Sep 19, 2017
@SparkQA
Copy link

SparkQA commented Sep 19, 2017

Test build #81922 has finished for PR 19273 at commit 72c5a8e.

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

@cloud-fan cloud-fan deleted the revert branch September 19, 2017 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants