Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20208
Thank you so much, @gatorsmile . Sure. I'll make a PR to improve error
handling for that.
---
-
To unsubscribe, e-mail: re
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20208
Also, thank you, @HyukjinKwon .
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20208
Thanks! Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20208
@dongjoon-hyun This PR is to improve the test coverage. LGTM.
When the schema do not match with the schemas of underlying data source,
the current error messages might be weird. This is
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20208
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92939/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20208
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/20208
**[Test build #92939 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92939/testReport)**
for PR 20208 at commit
[`a7064ac`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20208
**[Test build #92939 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92939/testReport)**
for PR 20208 at commit
[`a7064ac`](https://github.com/apache/spark/commit/a7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20208
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/20208
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/901/
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20208
@gatorsmile . Please let me know if I need to do more.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
F
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20208
Retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20208
The test suite is designed like the following according to their features.
```scala
class CSVReadSchemaSuite
extends ReadSchemaSuite
with IntegralTypeTest
with ToDouble
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20208
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/20208
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/20208
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92828/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20208
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92827/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20208
**[Test build #92827 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92827/testReport)**
for PR 20208 at commit
[`767d7ba`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20208
**[Test build #92828 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92828/testReport)**
for PR 20208 at commit
[`a7064ac`](https://github.com/apache/spark/commit/a
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20208
@gatorsmile , @HyukjinKwon .
Could you review this again for Spark 2.4?
---
-
To unsubscribe, e-mail: reviews-unsubscr.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20208
**[Test build #92828 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92828/testReport)**
for PR 20208 at commit
[`a7064ac`](https://github.com/apache/spark/commit/a7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20208
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/20208
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/829/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20208
**[Test build #92827 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92827/testReport)**
for PR 20208 at commit
[`767d7ba`](https://github.com/apache/spark/commit/76
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20208
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/828/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20208
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
26 matches
Mail list logo