Skip to content

[SPARK-26010][R] fix vignette eval with Java 11#23007

Closed
felixcheung wants to merge 5 commits intoapache:masterfrom
felixcheung:rjavavervig
Closed

[SPARK-26010][R] fix vignette eval with Java 11#23007
felixcheung wants to merge 5 commits intoapache:masterfrom
felixcheung:rjavavervig

Conversation

@felixcheung
Copy link
Member

What changes were proposed in this pull request?

changes in vignette only to disable eval

How was this patch tested?

Jenkins

@felixcheung felixcheung changed the title [SPARK-26010] fix vignette eval with Java 11 [SPARK-26010][R] fix vignette eval with Java 11 Nov 11, 2018
@SparkQA
Copy link

SparkQA commented Nov 11, 2018

Test build #98704 has finished for PR 23007 at commit dc4ba6c.

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

@SparkQA
Copy link

SparkQA commented Nov 11, 2018

Test build #98705 has finished for PR 23007 at commit 612b594.

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

@SparkQA
Copy link

SparkQA commented Nov 11, 2018

Test build #98706 has finished for PR 23007 at commit 63e3b6f.

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

@SparkQA
Copy link

SparkQA commented Nov 11, 2018

Test build #98707 has finished for PR 23007 at commit 385eb3f.

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

@SparkQA
Copy link

SparkQA commented Nov 11, 2018

Test build #98708 has finished for PR 23007 at commit 6dd4329.

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

asfgit pushed a commit that referenced this pull request Nov 13, 2018
## What changes were proposed in this pull request?

changes in vignette only to disable eval

## How was this patch tested?

Jenkins

Author: Felix Cheung <felixcheung_m@hotmail.com>

Closes #23007 from felixcheung/rjavavervig.

(cherry picked from commit 88c8262)
Signed-off-by: Felix Cheung <felixcheung@apache.org>
@felixcheung
Copy link
Member Author

merged to master/2.4

@asfgit asfgit closed this in 88c8262 Nov 13, 2018
@felixcheung
Copy link
Member Author

merged to branch-2.3 also (since we are making 2.3.3)

@felixcheung
Copy link
Member Author

@maropu could you pick up this for 2.3.3? looks like I've just missed by 15min

@maropu
Copy link
Member

maropu commented Jan 13, 2019

@felixcheung Thanks, pinging me! Yea, I'll do that.

asfgit pushed a commit that referenced this pull request Jan 13, 2019
## What changes were proposed in this pull request?

changes in vignette only to disable eval

## How was this patch tested?

Jenkins

Author: Felix Cheung <felixcheung_m@hotmail.com>

Closes #23007 from felixcheung/rjavavervig.

(cherry picked from commit 88c8262)
Signed-off-by: Felix Cheung <felixcheung@apache.org>
jackylee-ch pushed a commit to jackylee-ch/spark that referenced this pull request Feb 18, 2019
## What changes were proposed in this pull request?

changes in vignette only to disable eval

## How was this patch tested?

Jenkins

Author: Felix Cheung <felixcheung_m@hotmail.com>

Closes apache#23007 from felixcheung/rjavavervig.
kai-chi pushed a commit to kai-chi/spark that referenced this pull request Jul 23, 2019
## What changes were proposed in this pull request?

changes in vignette only to disable eval

## How was this patch tested?

Jenkins

Author: Felix Cheung <felixcheung_m@hotmail.com>

Closes apache#23007 from felixcheung/rjavavervig.

(cherry picked from commit 88c8262)
Signed-off-by: Felix Cheung <felixcheung@apache.org>
kai-chi pushed a commit to kai-chi/spark that referenced this pull request Aug 1, 2019
## What changes were proposed in this pull request?

changes in vignette only to disable eval

## How was this patch tested?

Jenkins

Author: Felix Cheung <felixcheung_m@hotmail.com>

Closes apache#23007 from felixcheung/rjavavervig.

(cherry picked from commit 88c8262)
Signed-off-by: Felix Cheung <felixcheung@apache.org>
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