Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/14803
I'm going to merge this one to branch 2.0 since it only changes Structured
Streaming.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14803
@marmbrus @frreiss @srowen @tdas Thanks for review!
---
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
Github user marmbrus commented on the issue:
https://github.com/apache/spark/pull/14803
Thanks, I'm going to merge this 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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14803
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/14803
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65812/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14803
**[Test build #65812 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65812/consoleFull)**
for PR 14803 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14803
**[Test build #65812 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65812/consoleFull)**
for PR 14803 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14803
>
* What error is printed (if any) if an invalid partition directory is
created midstream.
The error is:
[info] org.apache.spark.sql.streaming.StreamingQueryException:
Github user marmbrus commented on the issue:
https://github.com/apache/spark/pull/14803
Mostly looks good, I've also asked @tdas to take a look since he wrote this
initially.
A few more cases came to mind while while I was rephrasing your
documentation. Specifically,
-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14803
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65764/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14803
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/14803
**[Test build #65764 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65764/consoleFull)**
for PR 14803 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14803
**[Test build #65764 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65764/consoleFull)**
for PR 14803 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14803
ping @marmbrus Can you take another look?
And I have a question about the suggested programming guide change. Can you
clarify it too? Thanks!
---
If your project is set up for it, you can
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14803
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65627/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14803
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/14803
**[Test build #65627 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65627/consoleFull)**
for PR 14803 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14803
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65625/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14803
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/14803
**[Test build #65625 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65625/consoleFull)**
for PR 14803 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14803
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/14803
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65623/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14803
**[Test build #65623 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65623/consoleFull)**
for PR 14803 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14803
@marmbrus
>
* I think that for all but text you have to include the partition columns
in the schema if inference is turned off (which it is by default).
For text format, when
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14803
**[Test build #65627 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65627/consoleFull)**
for PR 14803 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14803
**[Test build #65625 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65625/consoleFull)**
for PR 14803 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14803
>
* If the partition directories are not present when the stream starts
then I believe this breaks.
Yes. Schema inference only happens when starting the stream.
>
* I
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14803
**[Test build #65623 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65623/consoleFull)**
for PR 14803 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14803
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/14803
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65595/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14803
**[Test build #65595 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65595/consoleFull)**
for PR 14803 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14803
I'm not so familiar with the logic here but tend to trust your analysis and
the other positive review.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14803
**[Test build #65595 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65595/consoleFull)**
for PR 14803 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14803
ping @marmbrus @zsxwing Would you mind to take a look this and provide your
feedback? If this is not going to be fixed, please let me know too. This is a
small change and I don't think it should be
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14803
ping @marmbrus @zsxwing Can you take a quick look? Thanks.
---
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
Github user frreiss commented on the issue:
https://github.com/apache/spark/pull/14803
LGTM
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14803
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64650/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14803
**[Test build #64650 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64650/consoleFull)**
for PR 14803 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14803
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64565/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14803
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/14803
**[Test build #64565 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64565/consoleFull)**
for PR 14803 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14803
**[Test build #64565 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64565/consoleFull)**
for PR 14803 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14803
ping @zsxwing @tdas Can you help review this? Thanks.
---
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/14803
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64405/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14803
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/14803
**[Test build #64405 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64405/consoleFull)**
for PR 14803 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14803
**[Test build #64405 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64405/consoleFull)**
for PR 14803 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14803
cc @marmbrus
---
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
48 matches
Mail list logo