Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15693
@hvanhovell OK. I will open a backport later.
---
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 featur
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/15693
LGTM. Merging to master/2.1. Thanks!
Could you open a backport for 2.0?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15693
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/15693
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68269/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15693
**[Test build #68269 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68269/consoleFull)**
for PR 15693 at commit
[`8b71061`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15693
**[Test build #68269 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68269/consoleFull)**
for PR 15693 at commit
[`8b71061`](https://github.com/apache/spark/commit/8
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15693
ping @hvanhovell Please let me know if you have other more comments. 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 pr
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15693
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68173/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15693
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/15693
**[Test build #68173 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68173/consoleFull)**
for PR 15693 at commit
[`448abfa`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15693
**[Test build #68173 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68173/consoleFull)**
for PR 15693 at commit
[`448abfa`](https://github.com/apache/spark/commit/4
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/15693
Thanks, good to see generated code for ease of understanding.
While a local variable `value4` at line 239 cannot be accessed before this
PR, an instance variable `classChildVar1` at line 306 can be
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15693
@kiszk the part of generated codes are posted as follows.
Before this patch:
Notice that line 239 `funcResult3 = value4._2();`, `value4` cannot be
accessed.
/* 288 */
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/15693
Would it be possible to put a part of generated code before and after this
PR?
---
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
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15693
@hvanhovell any more thoughts?
---
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 w
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15693
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67850/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15693
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/15693
**[Test build #67850 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67850/consoleFull)**
for PR 15693 at commit
[`5006bcb`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15693
**[Test build #67850 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67850/consoleFull)**
for PR 15693 at commit
[`5006bcb`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15693
**[Test build #67811 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67811/consoleFull)**
for PR 15693 at commit
[`0b660e0`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15693
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/15693
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67811/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15693
**[Test build #67811 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67811/consoleFull)**
for PR 15693 at commit
[`0b660e0`](https://github.com/apache/spark/commit/0
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15693
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, o
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15693
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/15693
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67808/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15693
**[Test build #67808 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67808/consoleFull)**
for PR 15693 at commit
[`0b660e0`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15693
**[Test build #67808 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67808/consoleFull)**
for PR 15693 at commit
[`0b660e0`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15693
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/15693
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67807/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15693
**[Test build #67807 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67807/consoleFull)**
for PR 15693 at commit
[`cb876e4`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15693
**[Test build #67807 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67807/consoleFull)**
for PR 15693 at commit
[`cb876e4`](https://github.com/apache/spark/commit/c
32 matches
Mail list logo