Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13680
how about `[offset1] [offset2] [not written, use offset3] [offset3]`? Then
we are still able to calculate the length by subtracting adjacent offsets.
---
If your project is set up for it, you can
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/13680
@cloud-fan Option 1 did not work for some cases. If
```UnsafeArrayWriter.write()``` is not called for all of elements, e.g. only
some of elements are written. For example, [this
code](https://github.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
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/13680
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61731/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61731 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61731/consoleFull)**
for PR 13680 at commit
[`1771ba3`](https://github.com/apache/spark/commit/
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/13680
Implemented 4-byte offset instead of 8-byte length&offset
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61731 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61731/consoleFull)**
for PR 13680 at commit
[`1771ba3`](https://github.com/apache/spark/commit/1
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/13680
Sorry. This is my misunderstanding. I was confused between ```UnsafeRow```
and ```UnsafeArrayData```. ```UnsafeArrayData``` keeps only one type in an
instance.
```[integer] [offset] [float] [offse
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/13680
Option 1 can work for this array: ```UnsafeDataArray: ...[integer] [offset]
[offset] [float]```. This is because 2 offsets are adjacent.
Can option 1 work for this ```UnsafeDataArray: ...[integer]
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13680
hmmm, looks like we are not in the same page... How could 2 offsets not
adjacent? We only keep offsets in the `value or offset` region, and put them
one by one.
---
If your project is set up for
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13680
One more thought about the format: `UnsafeRow` use 8 bytes to store offset
and length for variable-length type, this is because `UnsafeRow` is
word-aligned, so we can't calculate the element size
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
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/13680
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61673/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61673 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61673/consoleFull)**
for PR 13680 at commit
[`243252a`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61673 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61673/consoleFull)**
for PR 13680 at commit
[`243252a`](https://github.com/apache/spark/commit/2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61671 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61671/consoleFull)**
for PR 13680 at commit
[`2cf96b4`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61671/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
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/13680
**[Test build #61671 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61671/consoleFull)**
for PR 13680 at commit
[`2cf96b4`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61660/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
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/13680
**[Test build #61660 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61660/consoleFull)**
for PR 13680 at commit
[`7576c19`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61660 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61660/consoleFull)**
for PR 13680 at commit
[`7576c19`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
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/13680
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61620/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61620 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61620/consoleFull)**
for PR 13680 at commit
[`6a8a97f`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61620 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61620/consoleFull)**
for PR 13680 at commit
[`6a8a97f`](https://github.com/apache/spark/commit/6
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13680
can we also rerun `UDTSerializationBenchmark`? I think it should be faster
now.
---
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 kiszk commented on the issue:
https://github.com/apache/spark/pull/13680
@cloud-fan , could you please review this again?
I added benchmark programs and their results, and addressed your review
comments.
---
If your project is set up for it, you can reply to this emai
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61379/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
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/13680
**[Test build #61379 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61379/consoleFull)**
for PR 13680 at commit
[`45d2ec2`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61379 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61379/consoleFull)**
for PR 13680 at commit
[`45d2ec2`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61377/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
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/13680
**[Test build #61377 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61377/consoleFull)**
for PR 13680 at commit
[`8b1cbbb`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61377 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61377/consoleFull)**
for PR 13680 at commit
[`8b1cbbb`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61362/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
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/13680
**[Test build #61362 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61362/consoleFull)**
for PR 13680 at commit
[`9c113aa`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61362 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61362/consoleFull)**
for PR 13680 at commit
[`9c113aa`](https://github.com/apache/spark/commit/9
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13680
looks pretty good, could you add a benchmark in this PR? an example is
https://github.com/apache/spark/pull/12640/files#diff-b118a818177121a108fa92d0354871a6R26
---
If your project is set up for
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/13680
@rxin thank you for your comment. As you said, holistic view is important.
This PR is not only for machine learning.
This PR has another use case for improving projection of an array in any
p
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/13680
@cloud-fan and @hvanhovell thank you for your comments. Based on your
comments, I implemented ```UnsafeArrayData``` by using one implementation with
explicit clearing ```null bits``` by ```Arrays.fill
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61239/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
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/13680
**[Test build #61239 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61239/consoleFull)**
for PR 13680 at commit
[`500e978`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61239 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61239/consoleFull)**
for PR 13680 at commit
[`500e978`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
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/13680
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61236/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61236 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61236/consoleFull)**
for PR 13680 at commit
[`138810b`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61236 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61236/consoleFull)**
for PR 13680 at commit
[`138810b`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
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/13680
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61235/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61235 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61235/consoleFull)**
for PR 13680 at commit
[`687be4d`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61235 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61235/consoleFull)**
for PR 13680 at commit
[`687be4d`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61233/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
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/13680
**[Test build #61233 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61233/consoleFull)**
for PR 13680 at commit
[`682c397`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61233 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61233/consoleFull)**
for PR 13680 at commit
[`682c397`](https://github.com/apache/spark/commit/6
101 - 160 of 160 matches
Mail list logo