Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135664705
[Test build #41735 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41735/consoleFull)
for PR 8343 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135663625
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135663634
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135663528
Jenkins, retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135667676
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135667678
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135667588
[Test build #41735 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41735/console)
for PR 8343 at commit
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135821208
lint-r looks fine, Merging this.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8343
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user lresende commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135572901
Thanks for the comments @shivaram and @sun-rui. I have standardized in
using supressWarnings for all invocations of normalizePath.
---
If your project is set up for
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135575531
[Test build #41709 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41709/console)
for PR 8343 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135572688
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135572665
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135573138
[Test build #41709 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41709/consoleFull)
for PR 8343 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135575627
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135575628
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135596210
[Test build #41715 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41715/consoleFull)
for PR 8343 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135598611
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135598213
[Test build #41715 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41715/console)
for PR 8343 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135598617
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135616422
@lresende I'm just retesting this as we merged a R style checker. I'm sure
this PR should be fine, but just want to run this through to make sure things
are working
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135616345
Jenkins, retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135617728
@yu-iskw I actually an error in Jenkins which says. I guess this is from
the PR that added na.omit to the NAMESPACE yesterday
```
Error in (function (classes,
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135620319
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135620315
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135619437
I think I found the problem. Our `setGeneric` for `na.omit` is wrong. It is
being too restrictive. We need a diff which looks like
```
-
Github user yu-iskw commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135617969
@shivaram I see. I'll investigate the cause. Thank you for letting me know.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135616551
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135616536
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135616119
Thanks @lresende -- LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135620132
[Test build #41718 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41718/console)
for PR 8343 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135595941
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135595930
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8343#issuecomment-135617402
[Test build #41718 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41718/consoleFull)
for PR 8343 at commit
34 matches
Mail list logo