Skip to content

Conversation

@shahidki31
Copy link
Contributor

What changes were proposed in this pull request?

Back port the commit #23181 into Spark2.4 branch

How was this patch tested?

Added UT

@shahidki31
Copy link
Contributor Author

Jenkins, test this please

@SparkQA
Copy link

SparkQA commented Dec 1, 2018

Test build #99535 has finished for PR 23191 at commit 590d580.

  • This patch fails Scala style tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@shahidki31
Copy link
Contributor Author

Jenkins, retest this please

@SparkQA
Copy link

SparkQA commented Dec 1, 2018

Test build #99537 has finished for PR 23191 at commit 8656936.

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

@shahidki31
Copy link
Contributor Author

cc @vanzin . Kindly review

@vanzin
Copy link
Contributor

vanzin commented Dec 3, 2018

Merging to 2.4. Please close the PR manually.

asfgit pushed a commit that referenced this pull request Dec 3, 2018
…or failure jobs in the history server UI

Back port the commit #23181 into Spark2.4 branch

Added UT

Closes #23191 from shahidki31/branch-2.4.

Authored-by: Shahid <[email protected]>
Signed-off-by: Marcelo Vanzin <[email protected]>
@shahidki31
Copy link
Contributor Author

Thanks a lot @vanzin @srowen

@shahidki31 shahidki31 closed this Dec 3, 2018
kai-chi pushed a commit to kai-chi/spark that referenced this pull request Jul 23, 2019
…or failure jobs in the history server UI

Back port the commit apache#23181 into Spark2.4 branch

Added UT

Closes apache#23191 from shahidki31/branch-2.4.

Authored-by: Shahid <[email protected]>
Signed-off-by: Marcelo Vanzin <[email protected]>
kai-chi pushed a commit to kai-chi/spark that referenced this pull request Aug 1, 2019
…or failure jobs in the history server UI

Back port the commit apache#23181 into Spark2.4 branch

Added UT

Closes apache#23191 from shahidki31/branch-2.4.

Authored-by: Shahid <[email protected]>
Signed-off-by: Marcelo Vanzin <[email protected]>
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.

4 participants