Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/16973
thanks for the review. merged 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 this
Github user thunterdb commented on the issue:
https://github.com/apache/spark/pull/16973
These changes look good to me, but my knowledge of R is very limited.
@mengxr should confirm.
---
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/16973
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73037/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16973
**[Test build #73037 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73037/testReport)**
for PR 16973 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16973
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/16973
**[Test build #73037 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73037/testReport)**
for PR 16973 at commit
