Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20658
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/20658
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/20658
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20658
**[Test build #87627 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87627/testReport)**
for PR 20658 at commit
[`d7e03cd`](https://github.com/apache/spark/commit/d7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20658
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/20658
**[Test build #87627 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87627/testReport)**
for PR 20658 at commit
[`d7e03cd`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20658
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87627/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20658
**[Test build #87647 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87647/testReport)**
for PR 20658 at commit
[`a49ffa0`](https://github.com/apache/spark/commit/a4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20658
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/20658
**[Test build #87647 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87647/testReport)**
for PR 20658 at commit
[`a49ffa0`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20658
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87647/
Test PASSed.
---
Github user drboyer commented on the issue:
https://github.com/apache/spark/pull/20658
@HyukjinKwon thanks for the review so far! Sorry for the delay, I somehow
missed the Python style output in the test logs earlier. How's this look now?
Can you elaborate more on "doctest" if
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20658
Actually, I think I need some more feedback about matching
https://github.com/apache/spark/pull/20658#discussion_r170003442.
@rxin, @hvanhovell and @cloud-fan (from checking the history
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20658
cc @ueshin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/20658
So it looks like adding the database field has been backed out for this
change, which is probably the right thing to do, if we want to make that change
we should do that in a separate JIRA & commit
Github user drboyer commented on the issue:
https://github.com/apache/spark/pull/20658
Thanks @holdenk, I can open a separate JIRA about the missing field in
`Function` if it seems worth fixing. It wasn't critical for me, I just happened
to notice while doing some testing so I include
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20658
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
17 matches
Mail list logo