Github user parente commented on the issue:
https://github.com/apache/spark/pull/18339
@holdenk I rebased the PR and I think it's good to go if you'd like to give
it another look.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98175/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18339
**[Test build #98175 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98175/testReport)**
for PR 18339 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98174/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18339
**[Test build #98174 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98174/testReport)**
for PR 18339 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18339
**[Test build #98175 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98175/testReport)**
for PR 18339 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18339
**[Test build #98174 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98174/testReport)**
for PR 18339 at commit
Github user parente commented on the issue:
https://github.com/apache/spark/pull/18339
@holdenk Took a note to look at this weekend.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/18339
Since @HyukjinKwon's concerns for this PR have been addressed if @parente
can update this to master would be lovely to get this in for 3+ since I'm
working on some multi-language pipeline stuff
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18339
Jenkins left a comment asking like "Can one of the admins verify this
patch?" again. I was thinking it's worth given your comment above so I
retriggered.
I am not sure why / when / who
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/18339
@HyukjinKwon what re-triggered your interest in this PR?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91604/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18339
**[Test build #91604 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91604/testReport)**
for PR 18339 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18339
**[Test build #91604 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91604/testReport)**
for PR 18339 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18339
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/18339
Lets see what @BryanCutler thinks
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83998/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18339
**[Test build #83998 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83998/testReport)**
for PR 18339 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18339
**[Test build #83998 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83998/testReport)**
for PR 18339 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18339
I am okay with going ahead @holdenk if you think it's okay anyway.
---
-
To unsubscribe, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18339
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/18339
Jenkins OK to test.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18339
Thanks for cc'ing me. To me, I think I can follow the discussion and the
motivation here but I think I am neutral (rather -0) as `launch_gateway` itself
looks undocumented and [the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81570/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18339
**[Test build #81570 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81570/testReport)**
for PR 18339 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18339
**[Test build #81570 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81570/testReport)**
for PR 18339 at commit
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/18339
Let's get some extra eyes on this, maybe @davies or @HyukjinKwon want to
take a quick look? I think it makes sense as an advanced developer API but I'm
open to other ideas.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81472/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18339
**[Test build #81472 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81472/testReport)**
for PR 18339 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18339
**[Test build #81472 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81472/testReport)**
for PR 18339 at commit
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/18339
Jenkins ok to test.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/18339
The approach taken in
https://github.com/maxpoint/spylon-kernel/blob/master/spylon_kernel/scala_interpreter.py#L73
is interesting (and definitely not supported) - so making it easier for
kernels
Github user parente commented on the issue:
https://github.com/apache/spark/pull/18339
Oh neat. #17298 looks similar to the approach we took in spylon-kernel to
launch with stdout/stderr pipes redirected to the parent process and threads to
read them
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/18339
This is interesting, I've got a similar approach I've been working on in
https://github.com/apache/spark/pull/17298 which has some issues inside of
PyPI. Would that suit your needs if I extended it
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18339
Can one of the admins verify this patch?
---
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
48 matches
Mail list logo