Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23130
We don't need to block it, but @MaxGekk if you have time, it would great to
answer https://github.com/apache/spark/pull/23130#issuecomment-442491582
thanks, merging to master!
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23130
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99465/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23130
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/23130
**[Test build #99465 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99465/testReport)**
for PR 23130 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23130
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23130
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/23130
**[Test build #99465 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99465/testReport)**
for PR 23130 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23130
I think this change makes sense, at least it's good for performance. My
only concern is, shall we ask all the parsers to return Nil for empty files?
AFAIK JSON doesn't follow it.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23130
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/23130
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99335/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23130
**[Test build #99335 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99335/testReport)**
for PR 23130 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23130
**[Test build #99335 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99335/testReport)**
for PR 23130 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23130
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23130
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23130
The code change LGTM. There is a mistake in PR description: we updated
`FileSourceScanExec` not `DataSourceScanExec`. Let's also mention that this
fixed a behavior change introduced by
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23130
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/23130
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99239/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23130
**[Test build #99239 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99239/testReport)**
for PR 23130 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23130
**[Test build #99239 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99239/testReport)**
for PR 23130 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23130
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23130
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/23130
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/23130
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99227/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23130
**[Test build #99227 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99227/testReport)**
for PR 23130 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23130
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23130
@cloud-fan @HyukjinKwon Please, take a look at the PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23130
**[Test build #99227 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99227/testReport)**
for PR 23130 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23130
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23130
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
29 matches
Mail list logo