-
Notifications
You must be signed in to change notification settings - Fork 28.6k
[DO-NOT-MERGE] Investigating SparkR test failure #20005
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Test build #85033 has finished for PR 20005 at commit
|
I've tried to investigate this and I think it is due to a wrongly formatted document at https://cran.r-project.org/src/contrib/PACKAGES.in:
There is an additional breaking line here and it breaks the function |
Note: I've emailed to cran sysadmin team and asked them for help to fix the broken page. |
I see. Nice to get some helps. BTW, this also seems related with specific R version as well. I wasn't able to reproduce this with R 3.4.0 but I am able to reproduce this with R 3.1.1 (same as Jenkins'). Probably, some corner cases were handled in |
I use R version 3.2.3 and I can reproduce this too. |
cc @shivaram, @felixcheung and @falaki too FYI. |
Oh, Thanks for cran sysadmin team's quick reply, they fixed that broken page. |
retest this please. |
@viirya you are my hero. |
Test build #85045 has finished for PR 20005 at commit
|
retest this please. |
Verified locally, this issue fixed. Let's wait for Jenkins. |
Just checked it passes with R 3.1.1 in my local too! |
Test build #85046 has finished for PR 20005 at commit
|
I think we can safely resolve https://issues.apache.org/jira/browse/SPARK-22812. Let me leave this PR closed as well. Thanks to @viirya and CRAN sysadmin team. |
Ok. tests passed. Looks good. |
Thank you guys. |
Nice thanks!
|
Thanks @viirya! |
What changes were proposed in this pull request?
Don't merge this. I would like to check few things. I am super busy for the rest of this month so I wouldn't make it shortly.
Please keep investigating this if anyone is doing.
How was this patch tested?
N/A