-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ARROW-3194: [JAVA] Use the split length for setting value count of target vector #2524
Conversation
@laurentgo or @jacques-n can you review? |
Ping, @siddharthteotia do you want to get this into the 0.11 release? |
if (this.valueCount > 0) { | ||
target.setValueCount(this.valueCount); | ||
} | ||
target.setValueCount(length); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't we still need the if the condition when the length is zero?
Author: Kouhei Sutou <kou@clear-code.com> Closes apache#2701 from kou/add-workaround-missing-gemfile and squashes the following commits: fc80d6e <Kouhei Sutou> Add workaround to verify 0.11.0
It's helpful to setup test environment. Author: Kouhei Sutou <kou@clear-code.com> Closes apache#2702 from kou/c-glib-add-missing-gemfile-to-archive and squashes the following commits: 5c97dad <Kouhei Sutou> Include Gemfile to archive
Author: Kouhei Sutou <kou@clear-code.com> Closes apache#2703 from kou/expand-variables-in-commit-message and squashes the following commits: da4f3bc <Kouhei Sutou> Expand variables in commit message
Author: Pindikura Ravindra <ravindra@dremio.com> Closes apache#2695 from pravindra/re2 and squashes the following commits: 4408b85 <Pindikura Ravindra> ARROW-3331: Add re2 to toolchain
6ef7ee8
to
c8402d7
Compare
Addressed review comment. |
Looks good. +1 |
Resolved in 074e2c2. I'm not sure why this was never merged before |
We need to use the split length as the value count of the target vector. We are incorrectly using the value count of the current vector for the target vector