Github user yu-iskw commented on the pull request:
https://github.com/apache/spark/pull/6947#issuecomment-116903439
@JoshRosen Thank you for merging it!
---
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/6947
---
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 enab
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/6947#issuecomment-116749219
Great; merging to master.
---
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 shivaram commented on the pull request:
https://github.com/apache/spark/pull/6947#issuecomment-116747612
Ah okay. In that case I'm fine with adding this. If the existing patch
fixes the problem feel free to merge it
---
If your project is set up for it, you can reply to t
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/6947#issuecomment-116747028
@shivaram, I think the motivation here is to address local `dev/run-tests`
failures that are caused by the presence of these documentation files:
```
[josh
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/6947#issuecomment-115009673
@yu-iskw These files are not actually a part of the repo right ? I see that
they are generated during the build but we don't ship them as source code, so
I'm not sure w
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6947#issuecomment-114409976
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6947#issuecomment-114409817
[Test build #35524 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35524/console)
for PR 6947 at commit
[`5ca240c`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6947#issuecomment-114376372
[Test build #35524 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35524/consoleFull)
for PR 6947 at commit
[`5ca240c`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6947#issuecomment-114376309
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 have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6947#issuecomment-114376302
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 h
Github user yu-iskw commented on the pull request:
https://github.com/apache/spark/pull/6947#issuecomment-114375941
@davies Could you review it please when you have time? Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
GitHub user yu-iskw opened a pull request:
https://github.com/apache/spark/pull/6947
[SPARK-8554] Add the SparkR document files to `.rat-excludes` for
`./dev/check-license`
[[SPARK-8554] Add the SparkR document files to `.rat-excludes` for
`./dev/check-license` - ASF
JIRA](https:/
13 matches
Mail list logo