Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20151
**[Test build #85677 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85677/testReport)**
for PR 20151 at commit
[`f74df4b`](https://github.com/apache/spark/commit/f7
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20151
@holdenk, @rxin, @joshrosen and @ueshin, as you all might already know, I
am working on Python coverage. Based on the top of this PR, I think we can
leave the main codes intact while we properl
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20151
**[Test build #85677 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85677/testReport)**
for PR 20151 at commit
[`f74df4b`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20151
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85677/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20151
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20151
Hey @rxin, I think I need your sign-off too as it's related with SPARK-7721.
---
-
To unsubscribe, e-mail: reviews-unsubscr..
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/20151
The changes LGTM.
Btw, what if we miss the module in python path? Can we see that the error
is caused by the missing module from the exception message?
---
-
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20151
I manually tested after setting `spark.python.daemon.module` to
`nonexistantmodule`. It shows the error message like this:
```python
>>> spark.range(1).rdd.map(lambda x: x).collect()
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/20151
Looks good. Let's wait for @rxin's response.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional com
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20151
Yup, thanks for all review @felixcheung and @ueshin BTW
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Fo
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20151
@rxin or @joshrosen could you guys take a quick look and see if it makes
sense?
---
-
To unsubscribe, e-mail: reviews-unsubs
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/20151
So I think this could be the basis for solving a lot of related problems
and I like the minimally invasive approach to it. I think the error message for
setting it to a bad module rather than a none
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20151
+1... this is "undocumented" conf, sooo it's an expert one :)
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20151
Yup, will write up some more warnings that says like it's expert only
configuration, experimental and rather an internal configuration. Also, I will
note that we should be super careful. Will up
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20151
**[Test build #85917 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85917/testReport)**
for PR 20151 at commit
[`ea5b987`](https://github.com/apache/spark/commit/ea
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20151
**[Test build #85918 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85918/testReport)**
for PR 20151 at commit
[`fc65803`](https://github.com/apache/spark/commit/fc
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20151
**[Test build #85917 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85917/testReport)**
for PR 20151 at commit
[`ea5b987`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20151
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85917/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20151
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/20151
**[Test build #85918 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85918/testReport)**
for PR 20151 at commit
[`fc65803`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20151
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85918/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20151
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20151
Will merge this one if there isn't any objection. I believe this doesn't
affect the existing code path anyway ..
---
-
To u
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20151
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20151
**[Test build #86075 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86075/testReport)**
for PR 20151 at commit
[`fc65803`](https://github.com/apache/spark/commit/fc
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20151
**[Test build #86075 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86075/testReport)**
for PR 20151 at commit
[`fc65803`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20151
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/20151
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86075/
Test PASSed.
---
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20151
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
29 matches
Mail list logo