Github user holdensmagicalunicorn commented on the issue:
https://github.com/apache/spark/pull/21946
@cloud-fan, thanks! I am a bot who has found some folks who might be able
to help with the review:@gatorsmile, @zsxwing and @tdas
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21946
**[Test build #93888 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93888/testReport)**
for PR 21946 at commit
[`19808d5`](https://github.com/apache/spark/commit/19
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21946
cc @rxin @rdblue @jose-torres
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-ma
Github user jose-torres commented on the issue:
https://github.com/apache/spark/pull/21946
Wouldn't the redo of the API that we're discussing obsolete this?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21946
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-unified/1568/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21946
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21946
In the new proposal, we just rename `ReadSupport` to
`BatchReadSupportProvider`, so this change is kind of part of the big proposal.
---
-
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/21946
Isn't this unnecessary after the API redesign?
For the redesign, the `DataSourceV2` or a `ReadSupportProvider` will supply
a `create` method (or `anonymousTable`) to return a `Table` that imp
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/21946
@cloud-fan, from your comment around the same time as mine, it sounds like
the confusion may just be in how you're updating the current API to the
proposed one. Can you post a migration plan? It soun
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21946
> a ReadSupportProvider will supply a create method (or anonymousTable) to
return a Table that implements ReadSupport...
I'd prefer the current proposal in
https://docs.google.com/documen
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21946
@rdblue the plan is, I will have a big PR that implements the redesign.
However, if there is something makes sense even without the redesign, we should
have a separated PR. I think merging `ReadSu
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21946
> ReadSupport and ReadSupportWithSchema -> BatchReadSupportProvider
DataSourceReader -> ReadSupport
Yea, this is what I'm doing in my local branch for the redesign. I'll push
it soon w
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/21946
Yeah, I'm fine with this, then.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21946
**[Test build #93891 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93891/testReport)**
for PR 21946 at commit
[`6cac2b5`](https://github.com/apache/spark/commit/6c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21946
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/21946
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-unified/1571/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21946
**[Test build #93896 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93896/testReport)**
for PR 21946 at commit
[`1f0c9a7`](https://github.com/apache/spark/commit/1f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21946
**[Test build #93897 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93897/testReport)**
for PR 21946 at commit
[`417930a`](https://github.com/apache/spark/commit/41
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21946
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/21946
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-unified/1577/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21946
**[Test build #93888 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93888/testReport)**
for PR 21946 at commit
[`19808d5`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21946
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93888/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21946
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21946
@rdblue This change is pretty isolated. It also LGTM to me.
Since you are fine about the change, I am assuming you are not blocking
this. I will merge this soon.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21946
**[Test build #93891 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93891/testReport)**
for PR 21946 at commit
[`6cac2b5`](https://github.com/apache/spark/commit/6
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/21946
+1
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.o
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21946
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93891/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21946
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/21946
**[Test build #93896 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93896/testReport)**
for PR 21946 at commit
[`1f0c9a7`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21946
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/21946
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93896/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21946
**[Test build #93897 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93897/testReport)**
for PR 21946 at commit
[`417930a`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21946
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93897/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21946
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
34 matches
Mail list logo