-
Notifications
You must be signed in to change notification settings - Fork 28k
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-9324][SPARK-9322][SPARK-9321][SPARKR] Some aliases for R-like functions in DataFrames #7764
Conversation
Test build #38941 has finished for PR 7764 at commit
|
length(columns(x)) | ||
}) | ||
|
||
#' Returns the dimentions (number for rows and columns) of a DataFrame |
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.
typo: number of
Jenkins, retest this please |
Test build #39123 has finished for PR 7764 at commit
|
@falaki the documentation builds are failing with an error message
|
Test build #39266 has finished for PR 7764 at commit
|
# | ||
#' @rdname rbind | ||
#' @aliases unionAll | ||
setMethod("rbind", |
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.
theres an extra line of space before this setMethod
, but I can fix this up during merge
Thanks @falaki for the update. LGTM. I had a minor style fix that I'll apply during the merge. |
Adds following aliases: