Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18790
Could you change the PR title to `[SPARK-21587][SS] Added filter pushdown
through watermarks`?
LGTM
---
If your project is set up for it, you can reply to this email and have your
rep
Github user joseph-torres commented on the issue:
https://github.com/apache/spark/pull/18790
Agreed. I've restricted this PR to just filter, since the original story
was about enabling partition pruning for filters above the watermark.
---
If your project is set up for it, you can re
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18790
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80434/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18790
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18790
**[Test build #80434 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80434/testReport)**
for PR 18790 at commit
[`4cae897`](https://github.com/apache/spark/commit/4
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18790
It sounds like we need to revisit all the optimizer rules for
`EventTimeWatermark`? We can do it in the separate PRs.
---
If your project is set up for it, you can reply to this email and have y
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18790
**[Test build #80434 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80434/testReport)**
for PR 18790 at commit
[`4cae897`](https://github.com/apache/spark/commit/4c
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/18790
retest this please
---
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 i
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/18790
Ok to test
On Aug 8, 2017 7:48 PM, "UCB AMPLab" wrote:
> Can one of the admins verify this patch?
>
> â
> You are receiving this because you commented.
> Reply to thi
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/18790
Ok to test
On Aug 8, 2017 7:48 PM, "UCB AMPLab" wrote:
> Can one of the admins verify this patch?
>
> â
> You are receiving this because you commented.
> Reply t
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18790
Can one of the admins verify this patch?
---
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 feat
Github user joseph-torres commented on the issue:
https://github.com/apache/spark/pull/18790
I'm told I can reopen this instead of making a new PR for the same branch.
Reopening with fixed commit history.
---
If your project is set up for it, you can reply to this email and have your
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18790
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80424/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18790
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18790
**[Test build #80424 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80424/testReport)**
for PR 18790 at commit
[`f0dd408`](https://github.com/apache/spark/commit/f
Github user joseph-torres commented on the issue:
https://github.com/apache/spark/pull/18790
Created https://github.com/apache/spark/pull/18889 with everything
cherrypicked into the right place.
---
If your project is set up for it, you can reply to this email and have your
reply app
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/18790
I think this file has a bunch of change that are unrelated to this goal.
You will probably have clean up your branch and open up this PR again.
---
If your project is set up for it, you can reply to t
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18790
**[Test build #80424 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80424/testReport)**
for PR 18790 at commit
[`f0dd408`](https://github.com/apache/spark/commit/f0
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/18790
retest this please
---
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 i
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18790
Merged build finished. Test FAILed.
---
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18790
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80365/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18790
**[Test build #80365 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80365/testReport)**
for PR 18790 at commit
[`f0dd408`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18790
**[Test build #80365 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80365/testReport)**
for PR 18790 at commit
[`f0dd408`](https://github.com/apache/spark/commit/f0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18790
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80289/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18790
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18790
**[Test build #80289 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80289/testReport)**
for PR 18790 at commit
[`8c73117`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18790
**[Test build #80289 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80289/testReport)**
for PR 18790 at commit
[`8c73117`](https://github.com/apache/spark/commit/8c
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18790
retest this please
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18790
Merged build finished. Test FAILed.
---
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18790
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80215/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18790
**[Test build #80215 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80215/testReport)**
for PR 18790 at commit
[`8c73117`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18790
**[Test build #80215 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80215/testReport)**
for PR 18790 at commit
[`8c73117`](https://github.com/apache/spark/commit/8c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18790
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18790
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80127/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18790
**[Test build #80127 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80127/testReport)**
for PR 18790 at commit
[`14fec98`](https://github.com/apache/spark/commit/1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18790
**[Test build #80127 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80127/testReport)**
for PR 18790 at commit
[`14fec98`](https://github.com/apache/spark/commit/14
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18790
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18790
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80101/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18790
**[Test build #80101 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80101/testReport)**
for PR 18790 at commit
[`0a837ae`](https://github.com/apache/spark/commit/0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18790
**[Test build #80101 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80101/testReport)**
for PR 18790 at commit
[`0a837ae`](https://github.com/apache/spark/commit/0a
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/18790
ok to test
---
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/18790
Can one of the admins verify this patch?
---
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 feat
42 matches
Mail list logo