-
Notifications
You must be signed in to change notification settings - Fork 28.5k
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-13296][SQL] Move UserDefinedFunction into sql.expressions. #11181
Conversation
mima will probably fail |
Test build #51179 has finished for PR 11181 at commit
|
* limitations under the License. | ||
*/ | ||
|
||
package org.apache.spark.sql.execution.python |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this file is just a simple copy/paste from python.scala.
Test build #51180 has finished for PR 11181 at commit
|
Test build #51198 has finished for PR 11181 at commit
|
Test build #51202 has finished for PR 11181 at commit
|
cc @davies for review |
Test build #51222 has finished for PR 11181 at commit
|
Test build #51231 has finished for PR 11181 at commit
|
@davies this did pass tests - just some lingering thing that made the timeout. |
LGTM |
BTW, these movement will make us harder to find out the history of the changes, i'd like to not do that if it's not too bad. |
Test build #2539 has finished for PR 11181 at commit
|
I've merged this in master. |
This pull request has the following changes:
Most of the diffs are just straight copy-paste.
Test