yordan-pavlov edited a comment on pull request #7798: URL: https://github.com/apache/arrow/pull/7798#issuecomment-667344474
@paddyhoran yes, you are right, I added a couple more tests for sliced arrays and they didn't pass so seeing that the PR was not yet merged I added a few small changes to (1) implement support for filtering of sliced / offset data arrays (2) return an error if the filter array is offset - I thought it better to make this obvious rather than silently process the offset filter array incorrectly I looked briefly into implementing support for offset filter arrays but I couldn't figure out how to do it quickly - I might have to take another look at the C++ implementation and submit a separate PR for that. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
