Skip to content

ARROW-6773: [C++] Fix filter kernel when filtering with a boolean Array slice - #5570

Closed
nealrichardson wants to merge 3 commits into
apache:masterfrom
nealrichardson:fix-filter-slice
Closed

ARROW-6773: [C++] Fix filter kernel when filtering with a boolean Array slice#5570
nealrichardson wants to merge 3 commits into
apache:masterfrom
nealrichardson:fix-filter-slice

Conversation

@nealrichardson

@nealrichardsonnealrichardson commented Oct 2, 2019

Copy link
Copy Markdown
Member

No description provided.

@github-actions

github-actionsBot commented Oct 2, 2019

Copy link
Copy Markdown

https://issues.apache.org/jira/browse/ARROW-6733

@nealrichardson

nealrichardson commented Oct 2, 2019

Copy link
Copy Markdown
MemberAuthor

Travis passing on my fork: https://travis-ci.org/nealrichardson/arrow/builds/592762538

@nealrichardson

nealrichardson commented Oct 3, 2019

Copy link
Copy Markdown
MemberAuthor

Appveyor failure is something with installing glib? https://ci.appveyor.com/project/ApacheSoftwareFoundation/arrow/builds/27837139/job/kcgd1nufrsgcjqf0

@bkietz

bkietz commented Oct 3, 2019

Copy link
Copy Markdown
Member

It does look unrelated. I'll try restarting it to see if it evaporates

@nealrichardson

nealrichardson commented Oct 3, 2019

Copy link
Copy Markdown
MemberAuthor

I already restarted it (well, requested to restart it, so now it's in the queue again)

@nealrichardson

nealrichardson commented Oct 3, 2019

Copy link
Copy Markdown
MemberAuthor

Appveyor failed again but this also seems to be failing on master. @kou do you know what's up with https://ci.appveyor.com/project/ApacheSoftwareFoundation/arrow/builds/27840001/job/tgnp57wisjvhyf9a?

An error occurred while installing glib2 (3.3.7), and Bundler cannot continue.
Make sure that `gem install glib2 -v '3.3.7' --source 'https://rubygems.org/'`
succeeds before bundling.

@kou

kou commented Oct 3, 2019

Copy link
Copy Markdown
Member

This is not a Apache Arrow problem.
This is a glib2 gem problem. I'll fix it soon. Please ignore this for now.

@kou

kou commented Oct 3, 2019

Copy link
Copy Markdown
Member

#5572 will solve this.

@kou

kou commented Oct 3, 2019

Copy link
Copy Markdown
Member

#5572 has been merged.
Rebasing will solve this problem.

@bkietz

bkietz commented Oct 3, 2019

Copy link
Copy Markdown
Member

In that case I'm merging this

@bkietzbkietz changed the title ARROW-6733: [C++] Fix filter kernel when filtering with a boolean Array sliceARROW-6773: [C++] Fix filter kernel when filtering with a boolean Array sliceOct 3, 2019
@bkietzbkietz closed this in f2e8f85Oct 3, 2019
kszucs pushed a commit that referenced this pull request Oct 5, 2019
@nealrichardson@kszucs
ARROW-6773: [C++] Fix filter kernel when filtering with a boolean Arr…
99db64c
…ay slice
Closes#5570 from nealrichardson/fix-filter-slice and squashes the following commits:
3a772fc <Neal Richardson> Lint again
f7e9475 <Neal Richardson> lint
96220d1 <Neal Richardson> Test and fix for bug in filter kernel with a slice
Authored-by: Neal Richardson <neal.p.richardson@gmail.com>
Signed-off-by: Benjamin Kietzman <bengilgit@gmail.com>
This was referenced Oct 3, 2019
Closed
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bkietzbkietzbkietz approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nealrichardson@bkietz@kou