-
Notifications
You must be signed in to change notification settings - Fork 28.4k
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
SPARK-6063 #4815
SPARK-6063 #4815
Conversation
Update to latest Spark master
Updating to latest Spark master
Can one of the admins verify this patch? |
LGTM |
1 similar comment
LGTM |
Test build #623 has started for PR 4815 at commit
|
Test build #623 has finished for PR 4815 at commit
|
test this please |
Test build #28091 has started for PR 4815 at commit
|
Test build #28091 has finished for PR 4815 at commit
|
Test FAILed. |
Test build #624 has started for PR 4815 at commit
|
Test build #624 has finished for PR 4815 at commit
|
Not sure why these unrelated tests are failing.... By the way, can you please add more of a title to the PR? |
I don't think it's necessary to wait on Jenkins. This doc change can't cause a problem. We can fix the title on merge too. I'll wait anyway for that, but figure one of us can just merge soon in any event. |
…n LDAModel.scala Remove unicode characters from MLlib file. Author: Michael Griffiths <msjgriffiths@gmail.com> Author: Griffiths, Michael (NYC-RPM) <michael.griffiths@reprisemedia.com> Closes #4815 from msjgriffiths/SPARK-6063 and squashes the following commits: bcd7de1 [Griffiths, Michael (NYC-RPM)] Change \u201D quote marks around 'theta' to standard single apostrophe (\x27) 38eb535 [Michael Griffiths] Merge pull request #2 from apache/master b08e865 [Michael Griffiths] Merge pull request #1 from apache/master
Remove unicode characters from MLlib file.