Github user poplav commented on the issue:
https://github.com/apache/spark/pull/17087
Hi, All. I am trying to get this included into Spark 2.1.1. I opened a PR
https://github.com/apache/spark/pull/18942.
---
If your project is set up for it, you can reply to this email and have you
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/17087
Hi, All.
I'm wondering if it is too late for Spark 2.2.0 to include this.
Is this too risky for that?
---
If your project is set up for it, you can reply to this email and have your
re
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/17087
LGTM. 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 an
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
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/17087
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76972/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #76972 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76972/testReport)**
for PR 17087 at commit
[`a5fd465`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #76972 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76972/testReport)**
for PR 17087 at commit
[`a5fd465`](https://github.com/apache/spark/commit/a5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
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/17087
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76949/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #76949 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76949/testReport)**
for PR 17087 at commit
[`3868bf5`](https://github.com/apache/spark/commit/3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #76949 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76949/testReport)**
for PR 17087 at commit
[`3868bf5`](https://github.com/apache/spark/commit/38
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76862/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
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/17087
**[Test build #76862 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76862/testReport)**
for PR 17087 at commit
[`1f19c80`](https://github.com/apache/spark/commit/1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #76862 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76862/testReport)**
for PR 17087 at commit
[`1f19c80`](https://github.com/apache/spark/commit/1f
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/17087
ping @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 t
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/17087
@marmbrus could you please take a look?
---
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
enabl
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75999/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
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/17087
**[Test build #75999 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75999/testReport)**
for PR 17087 at commit
[`8b6ba75`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #75999 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75999/testReport)**
for PR 17087 at commit
[`8b6ba75`](https://github.com/apache/spark/commit/8b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
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/17087
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75961/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #75961 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75961/testReport)**
for PR 17087 at commit
[`c2e6b8c`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #75961 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75961/testReport)**
for PR 17087 at commit
[`c2e6b8c`](https://github.com/apache/spark/commit/c2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75951/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
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/17087
**[Test build #75951 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75951/testReport)**
for PR 17087 at commit
[`ea67c8a`](https://github.com/apache/spark/commit/e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #75951 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75951/testReport)**
for PR 17087 at commit
[`ea67c8a`](https://github.com/apache/spark/commit/ea
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #75950 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75950/testReport)**
for PR 17087 at commit
[`51a71a6`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75950/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #75950 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75950/testReport)**
for PR 17087 at commit
[`51a71a6`](https://github.com/apache/spark/commit/51
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #75948 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75948/testReport)**
for PR 17087 at commit
[`8548b0e`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75948/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #75948 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75948/testReport)**
for PR 17087 at commit
[`8548b0e`](https://github.com/apache/spark/commit/85
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/17087
@marmbrus could you please take a look?
---
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
enabl
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75487/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
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/17087
**[Test build #75487 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75487/testReport)**
for PR 17087 at commit
[`81545d6`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #75487 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75487/testReport)**
for PR 17087 at commit
[`81545d6`](https://github.com/apache/spark/commit/81
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/17087
@marmbrus could you please take a look?
---
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
enabl
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/17087
ping @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 t
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/17087
ping @marmbrus, @davies
---
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 kiszk commented on the issue:
https://github.com/apache/spark/pull/17087
ping @marmbrus, @davies
---
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 s
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/17087
@marmbrus, could you please take a look at this PR for fixing a problem?
---
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
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/17087
ping @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 t
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
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/17087
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74464/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #74464 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74464/testReport)**
for PR 17087 at commit
[`7db9aea`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #74464 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74464/testReport)**
for PR 17087 at commit
[`7db9aea`](https://github.com/apache/spark/commit/7d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
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/17087
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74442/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #74442 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74442/testReport)**
for PR 17087 at commit
[`530f84f`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74435/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #74442 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74442/testReport)**
for PR 17087 at commit
[`530f84f`](https://github.com/apache/spark/commit/53
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #74435 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74435/testReport)**
for PR 17087 at commit
[`11f56d1`](https://github.com/apache/spark/commit/1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #74435 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74435/testReport)**
for PR 17087 at commit
[`11f56d1`](https://github.com/apache/spark/commit/11
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/17087
Jenkins, 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 wishe
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
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/17087
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74428/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #74428 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74428/testReport)**
for PR 17087 at commit
[`11f56d1`](https://github.com/apache/spark/commit/11
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
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/17087
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74374/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #74374 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74374/testReport)**
for PR 17087 at commit
[`c5fc5f1`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #74374 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74374/testReport)**
for PR 17087 at commit
[`c5fc5f1`](https://github.com/apache/spark/commit/c5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
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/17087
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74313/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #74313 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74313/testReport)**
for PR 17087 at commit
[`a2f85cd`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #74313 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74313/testReport)**
for PR 17087 at commit
[`a2f85cd`](https://github.com/apache/spark/commit/a2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74311/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #74311 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74311/testReport)**
for PR 17087 at commit
[`c02589c`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
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/17087
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
enabled
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #74310 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74310/testReport)**
for PR 17087 at commit
[`5fb413f`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74310/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #74311 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74311/testReport)**
for PR 17087 at commit
[`c02589c`](https://github.com/apache/spark/commit/c0
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/17087
Thank you for pointing out `InterpretedPredicate`. Now, `newPredicate`
always returns `Predicate` that can be executed by calling `eval()`.
It looks simpler and better than the first implementation
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #74310 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74310/testReport)**
for PR 17087 at commit
[`5fb413f`](https://github.com/apache/spark/commit/5f
Github user marmbrus commented on the issue:
https://github.com/apache/spark/pull/17087
There appears to have been some code drift (as `GeneratePredicate` and
`InterpretedPredicate` both used to return a class that inherited from a common
interface), but I don't think its hard to just
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
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/17087
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74262/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #74262 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74262/testReport)**
for PR 17087 at commit
[`0e2bbe7`](https://github.com/apache/spark/commit/0
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/17087
@marmbrus I have just commit the code of intermediate refactoring. Would it
be possible to give comments?
If it is fine (return `null` and check it at caller), I will update other
caller sites for
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #74262 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74262/testReport)**
for PR 17087 at commit
[`0e2bbe7`](https://github.com/apache/spark/commit/0e
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/17087
I am refactoring `newPredicate`. `newPredicate` will catch the exception
and log a warning. However, I think that `newPredicate` cannot return an
interpreted result. This is because code generation re
Github user marmbrus commented on the issue:
https://github.com/apache/spark/pull/17087
I don't think we need a complex refactoring. Why can't `newPredicate`
catch the exception, log a warning and return an interpreted `Predicate`?
---
If your project is set up for it, you can reply
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/17087
@marmbrus thank you for your comments.
For feedback mechanism, I imagine that you are talking about
[this](https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/
Github user marmbrus commented on the issue:
https://github.com/apache/spark/pull/17087
I agree with the general approach of having a fallback from code generation
to interpreted evaluation, but I also agree that this feels too narrowly
targeted. In particular, why do this in one ope
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74019/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17087
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/17087
**[Test build #74019 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74019/testReport)**
for PR 17087 at commit
[`98cd961`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #74019 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74019/testReport)**
for PR 17087 at commit
[`98cd961`](https://github.com/apache/spark/commit/98
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/17087
I already identified where 64k limit is exceeded. As a result, I think that
it is not easy to fix the issue.
The previous issues are derived from independent blocks (in
`Seq[ExprCode]`) that can
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/17087
Is it right to just do this in one code path? why not all similar cases
where the 64k limit is exceeded?
Can the underlying problem be fixed or improved?
You're also catching all Exceptions, w
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/17087
@davies, could you please review this?
---
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
enable
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #73530 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73530/testReport)**
for PR 17087 at commit
[`6f40a93`](https://github.com/apache/spark/commit/6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17087
**[Test build #73530 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73530/testReport)**
for PR 17087 at commit
[`6f40a93`](https://github.com/apache/spark/commit/6f
100 matches
Mail list logo
