Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/18025
merged to master, thanks!
---
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
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/18025
AppVeyor failure is unfortunate. but it passed before a doc only change.
---
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 felixcheung commented on the issue:
https://github.com/apache/spark/pull/18025
haha. I like the `\emph`
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78264/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #78264 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78264/testReport)**
for PR 18025 at commit
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/18025
OK. Updated the doc for the cov method for SparkDataFrame.
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #78264 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78264/testReport)**
for PR 18025 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/18025
very cool, thanks, I guess there's only this last comment
https://github.com/apache/spark/pull/18025#discussion_r122631306
---
If your project is set up for it, you can reply to this email and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78244/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #78244 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78244/testReport)**
for PR 18025 at commit
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/18025
This is how the doc for column_aggregate_functions looks like (only
snapshot of the main parts):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #78244 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78244/testReport)**
for PR 18025 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78216/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #78216 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78216/testReport)**
for PR 18025 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #78216 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78216/testReport)**
for PR 18025 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78215/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
Merged build finished. Test FAILed.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #78215 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78215/testReport)**
for PR 18025 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #78215 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78215/testReport)**
for PR 18025 at commit
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/18025
@HyukjinKwon Thanks for catching this. They were incorrectly labeled as
math functions instead of aggregate functions in SparkR. And that's why I did
not change them.
New commit fixed
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18025
@actuaryzhang, Are `corr`, `covar_pop` and `covar_samp` the same instances
but missed? It looks these are also aggregate functions in `functions.scala`
but these look missed here and I can see
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/18025
@felixcheung Could you take another look and let me know if there is
anything else needed?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78190/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #78190 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78190/testReport)**
for PR 18025 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #78190 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78190/testReport)**
for PR 18025 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #78153 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78153/testReport)**
for PR 18025 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
Merged build finished. Test FAILed.
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78153/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
Merged build finished. Test FAILed.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #78152 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78152/testReport)**
for PR 18025 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78152/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #78149 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78149/testReport)**
for PR 18025 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78149/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #78153 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78153/testReport)**
for PR 18025 at commit
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/18025
@felixcheung Your comments are all addressed now. Please let me know if
there is anything else needed.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #78152 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78152/testReport)**
for PR 18025 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #78149 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78149/testReport)**
for PR 18025 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #77631 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77631/testReport)**
for PR 18025 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77631/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #77631 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77631/testReport)**
for PR 18025 at commit
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/18025
@HyukjinKwon Thanks much for the review. New commit now fixes the issues
you pointed out.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18025
BTW, I checked the description/examples. These generally look good to me.
---
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 HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18025
Will give a shot to help double check examples and contents about
correctness and consistency at my best tonight too.
---
If your project is set up for it, you can reply to this email and have
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/18025
Thanks for the update. Look forward to your feedback.
---
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 felixcheung commented on the issue:
https://github.com/apache/spark/pull/18025
thanks! sorry about the delay - this is very important to have (and among
your other pending PRs) - will pick this up again in ~ a week
---
If your project is set up for it, you can reply to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77420/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #77420 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77420/testReport)**
for PR 18025 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #77420 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77420/testReport)**
for PR 18025 at commit
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/18025
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 have this feature
enabled
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77414/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
Merged build finished. Test FAILed.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #77414 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77414/testReport)**
for PR 18025 at commit
59 matches
Mail list logo