mapleFU commented on issue #37559: URL: https://github.com/apache/arrow/issues/37559#issuecomment-1711067991
Ooops, sorry for misunderstand the logic here. After take a glance at the paper, seems it combine `selector` and the decode logic, and make it able to make the following op fast: ``` decoder.DecodeSpace(num_rows, selector, ...); ``` This also means putting `selector` or `DecodeSpace` support under arrow decoder internal, rather than calling a `Skip` above. I think this might take some time to stablize and testing it, since it would be really bug-prone. But I appreciate the way -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
