-
Notifications
You must be signed in to change notification settings - Fork 28.1k
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
[SPARKR] [SPARK-10328] Fix generic for na.omit #8495
Conversation
cc @yu-iskw |
@yu-iskw I also found a minor bug in lint-r that I just fixed. Please let me know if that is good. With this change lint-r passes on my machine |
@shivaram All right. I am checking this on my local machine. Please give me some minutes. |
@@ -413,7 +413,7 @@ setGeneric("dropna", | |||
#' @rdname nafunctions | |||
#' @export | |||
setGeneric("na.omit", |
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.
I'm just testing if we need this generic or not.
Add tests to check `na.omit` on a DataFrame
LGTM |
Cool - lintr checks also seem to be passing now. I'll merge this once tests pass |
Test build #41722 has finished for PR 8495 at commit
|
Test build #41724 has finished for PR 8495 at commit
|
S3 function is at https://stat.ethz.ch/R-manual/R-patched/library/stats/html/na.fail.html Author: Shivaram Venkataraman <shivaram@cs.berkeley.edu> Author: Shivaram Venkataraman <shivaram.venkataraman@gmail.com> Author: Yu ISHIKAWA <yuu.ishikawa@gmail.com> Closes #8495 from shivaram/na-omit-fix. (cherry picked from commit 2f99c37) Signed-off-by: Shivaram Venkataraman <shivaram@cs.berkeley.edu>
S3 function is at https://stat.ethz.ch/R-manual/R-patched/library/stats/html/na.fail.html