Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16680
Thank you so much.
---
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 feature
enabled and wishes
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16680
thanks, merging to master!
---
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 feature
enabled and
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16680
@cloud-fan, I just built it with 2.10 and checked it does not touch
original code path, line by line at my best.
---
If your project is set up for it, you can reply to this email and have your
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16680
Yes, I tried to only move and does not touch the code path at my best. Let
me check this again tomorrow with Scala 2.10 build and ping you again.
---
If your project is set up for it, you can
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16680
Sorry I didn't compare the CSV part with JSON part closely, I'm ok to keep
them consistent now. To confirm, this PR just moves codes around right?
---
If your project is set up for it, you can
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16680
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72446/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16680
Merged build finished. Test PASSed.
---
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 feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16680
**[Test build #72446 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72446/testReport)**
for PR 16680 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16680
**[Test build #72446 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72446/testReport)**
for PR 16680 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16680
(I am fine with changing the name only for CSV ones for now as well. I
would appreciate if you confirm please)
---
If your project is set up for it, you can reply to this email and have your
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16680
@cloud-fan, I just mainly resembled ones in JSON datasource and I am pretty
sure you knew this when you added some comments. But let me just rebase this as
is for now just in case maybe you are
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16680
Merged build finished. Test PASSed.
---
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 feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16680
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72024/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16680
**[Test build #72024 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72024/testReport)**
for PR 16680 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16680
**[Test build #72024 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72024/testReport)**
for PR 16680 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16680
Sure!
---
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 feature
enabled and wishes so, or if the
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16680
shall we rename `InferSchema` to `JsonInferSchema`?
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16680
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71967/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16680
Merged build finished. Test PASSed.
---
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 feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16680
**[Test build #71967 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71967/testReport)**
for PR 16680 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16680
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71966/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16680
Merged build finished. Test PASSed.
---
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 feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16680
**[Test build #71966 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71966/testReport)**
for PR 16680 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16680
@cloud-fan, could you please take a look? I tried to not change the current
behaviour and logics at my best but just re-locate them here.
---
If your project is set up for it, you can reply to
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16680
Let me double check tomorrow in case and then cc someone.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16680
**[Test build #71845 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71845/testReport)**
for PR 16680 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16680
**[Test build #71844 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71844/testReport)**
for PR 16680 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16680
**[Test build #71843 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71843/testReport)**
for PR 16680 at commit
28 matches
Mail list logo
