Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20204
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20204
Will merge this one if there's no more comments in few days.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.o
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20204
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86310/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20204
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/20204
**[Test build #86310 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86310/testReport)**
for PR 20204 at commit
[`df9af13`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20204
**[Test build #86310 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86310/testReport)**
for PR 20204 at commit
[`df9af13`](https://github.com/apache/spark/commit/df
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20204
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20204
Hi @rxin, @felixcheung, @ueshin, @holdenk and @JoshRosen. Could you take
another look please when you guys have some time?
@yhuai, I believe this is nice to have to prevent the previous
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20204
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86096/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20204
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/20204
**[Test build #86096 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86096/testReport)**
for PR 20204 at commit
[`df9af13`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20204
**[Test build #86096 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86096/testReport)**
for PR 20204 at commit
[`df9af13`](https://github.com/apache/spark/commit/df
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20204
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20204
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85994/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20204
**[Test build #85994 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85994/testReport)**
for PR 20204 at commit
[`e8e7112`](https://github.com/apache/spark/commit/e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20204
**[Test build #85994 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85994/testReport)**
for PR 20204 at commit
[`e8e7112`](https://github.com/apache/spark/commit/e8
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20204
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20204
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20204
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85969/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20204
**[Test build #85969 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85969/testReport)**
for PR 20204 at commit
[`e8e7112`](https://github.com/apache/spark/commit/e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20204
**[Test build #85969 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85969/testReport)**
for PR 20204 at commit
[`e8e7112`](https://github.com/apache/spark/commit/e8
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/20204
> @icexelloss, for #20204 (comment), yes, that's the way I usually use too.
My worry is though I wonder if this is a proper official way to do it because I
have been thinking this way is rather m
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20204
> looks reasonable to me.
> how are we going to track changes to coverage number?
Yea, that's a good point. I would like to do it too because other tools
allow it. I think this is an
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20204
@icexelloss, for
https://github.com/apache/spark/pull/20204#issuecomment-356431026, yes, that's
the way I usually use too. My worry is though I wonder if this is a proper
official way to do it
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20204
If anyone is worried of elapsed time, I think seems generally fine:
Before:
```
sh run-tests --python-executables=python3 --modules=pyspark-sql
```
```
...
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/20204
I usually run tests like this:
```
SPARK_TESTING=1 bin/pyspark pyspark.sql.tests ...
```
Does it make sense to expose similar usage for enabling coverage? Sth like
```
SPARK
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20204
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85862/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20204
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/20204
**[Test build #85862 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85862/testReport)**
for PR 20204 at commit
[`dadcae4`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20204
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20204
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85858/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20204
**[Test build #85858 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85858/testReport)**
for PR 20204 at commit
[`3c3c3cb`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20204
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85855/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20204
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/20204
**[Test build #85855 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85855/testReport)**
for PR 20204 at commit
[`a3179d7`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20204
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85856/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20204
Merged build finished. Test FAILed.
---
-
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/20204
**[Test build #85856 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85856/testReport)**
for PR 20204 at commit
[`9f2c400`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20204
**[Test build #85862 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85862/testReport)**
for PR 20204 at commit
[`dadcae4`](https://github.com/apache/spark/commit/da
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20204
Sorry for noice. I made some clean up and just rebased.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Fo
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20204
**[Test build #85858 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85858/testReport)**
for PR 20204 at commit
[`3c3c3cb`](https://github.com/apache/spark/commit/3c
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/20204
This is awesome. Thanks @HyukjinKwon!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comman
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20204
**[Test build #85856 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85856/testReport)**
for PR 20204 at commit
[`9f2c400`](https://github.com/apache/spark/commit/9f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20204
**[Test build #85855 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85855/testReport)**
for PR 20204 at commit
[`a3179d7`](https://github.com/apache/spark/commit/a3
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20204
cc @icexelloss too. I think we had a small talk about this before.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.ap
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20204
cc @rxin, @felixcheung, @ueshin, @holdenk and @JoshRosen. Here, I made it
based on #20151. Could you take a look for this one too? I can make this PR
separate from #20151 by manually adding the
46 matches
Mail list logo