Skip to content

Update for spark 2.2.2 release - #21707

Closed
tgravescs wants to merge 2 commits into
apache:branch-2.2from
tgravescs:spark222-release
Closed

Update for spark 2.2.2 release#21707
tgravescs wants to merge 2 commits into
apache:branch-2.2from
tgravescs:spark222-release

Conversation

@tgravescs

Copy link
Copy Markdown
Contributor

Release process for Spark 2.2.2 say to update the test.

@SparkQA

Copy link
Copy Markdown

Test build #92576 has finished for PR 21707 at commit cd9bdca.

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

@tgravescs

Copy link
Copy Markdown
ContributorAuthor

test this please

@SparkQA

Copy link
Copy Markdown

Test build #92593 has finished for PR 21707 at commit cd9bdca.

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

@tgravescs

Copy link
Copy Markdown
ContributorAuthor

@vanzin@gatorsmile

object PROCESS_TABLES extends QueryTest with SQLTestUtils {
// Tests the latest version of every release line.
val testingVersions = Seq("2.0.2", "2.1.2", "2.2.1")
val testingVersions = Seq("2.0.2", "2.1.2", "2.2.2")

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.

@tgravescs . Could you replace 2.1.2 with 2.1.3, too?

cc @vanzin (2.1.3 release manager).

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

updated

@SparkQA

Copy link
Copy Markdown

Test build #92759 has finished for PR 21707 at commit c0c2613.

  • 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.

Thank you. +1, LGTM.

@HyukjinKwon

Copy link
Copy Markdown
Member

retest this please

@SparkQA

Copy link
Copy Markdown

Test build #93040 has finished for PR 21707 at commit c0c2613.

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

@HyukjinKwon

Copy link
Copy Markdown
Member

oops, branch-2.2 doesn't have the fix yet https://github.com/apache/spark/pull/21779.

@HyukjinKwon

Copy link
Copy Markdown
Member

retest this please

@SparkQA

Copy link
Copy Markdown

Test build #93081 has started for PR 21707 at commit c0c2613.

@HyukjinKwon

Copy link
Copy Markdown
Member

retest this please

@SparkQA

Copy link
Copy Markdown

Test build #93090 has finished for PR 21707 at commit c0c2613.

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

@tgravescs

Copy link
Copy Markdown
ContributorAuthor

thanks for catching that, looks good now so merging this.

@tgravescs

Copy link
Copy Markdown
ContributorAuthor

merged to branch-2.2

asfgit pushed a commit that referenced this pull request Jul 16, 2018
Release process for Spark 2.2.2 say to update the test.
Author: Thomas Graves <tgraves@thirteenroutine.corp.gq1.yahoo.com>
Closes#21707 from tgravescs/spark222-release.
MatthewRBruce pushed a commit to Shopify/spark that referenced this pull request Jul 31, 2018
Release process for Spark 2.2.2 say to update the test.
Author: Thomas Graves <tgraves@thirteenroutine.corp.gq1.yahoo.com>
Closesapache#21707 from tgravescs/spark222-release.
Willymontaz pushed a commit to criteo-forks/spark that referenced this pull request Sep 26, 2019
Release process for Spark 2.2.2 say to update the test.
Author: Thomas Graves <tgraves@thirteenroutine.corp.gq1.yahoo.com>
Closesapache#21707 from tgravescs/spark222-release.
Willymontaz pushed a commit to criteo-forks/spark that referenced this pull request Sep 27, 2019
Release process for Spark 2.2.2 say to update the test.
Author: Thomas Graves <tgraves@thirteenroutine.corp.gq1.yahoo.com>
Closesapache#21707 from tgravescs/spark222-release.
Sign up for freeto 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.

5 participants

@tgravescs@SparkQA@HyukjinKwon@felixcheung@dongjoon-hyun