Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20501
ping @rekhajoshm ?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20501
can you reopen this PR and add [R] in the title?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20501
@rekhajoshm @felixcheung since 2.3.0 was just finished, OK to merge?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apach
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20501
@rekhajoshm feel free to follow up after we are through with 2.3.0, thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...
Github user rekhajoshm commented on the issue:
https://github.com/apache/spark/pull/20501
Ack. thanks for the update @felixcheung @srowen Closing this.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.o
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20501
It was not used in the current way we have the export and a change in 6.x
started warning about this.
Jenkins is not running Roxygen 6 though. There are pending tasks to
upgrade. IMO it
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20501
I see this too and Roxygen 6.0.1 seems common now. Is there any value to
the tags? I don't think these removals can cause much merge conflict. It can go
in master right?
---
--
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20501
I'm/We're aware of this and to keep diff small and manageable for
backporting fixes, it is preferable to avoid broad changes like this.
We can revisit when new releases are stable down,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20501
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87040/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20501
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20501
**[Test build #87040 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87040/testReport)**
for PR 20501 at commit
[`7db58db`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20501
**[Test build #87040 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87040/testReport)**
for PR 20501 at commit
[`7db58db`](https://github.com/apache/spark/commit/7d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20501
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/569/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20501
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
14 matches
Mail list logo