pearu commented on issue #50842: URL: https://github.com/apache/arrow/issues/50842#issuecomment-5281910027
Thanks for the offer, and good to have a second voice for (b) — that's the direction I'd settled on too. I am planning to implement it, so I'll take this one. The prerequisite fix (#50841) has landed, so the path is clear. If you'd like something adjacent to pick up: (a) is still potentially worth doing as a complement, for code paths that genuinely need a dense array and so aren't covered by (b) — `strings_to_categorical` being the obvious one. That's separable from this issue. Otherwise, review on the PR once it's up would be very welcome. Worth noting no maintainer has commented on the direction yet, so (b) is still two contributors' preference rather than a settled decision — input from an Arrow maintainer would be welcome before I get too far in. --- _🤖 Drafted by Claude Code (an AI agent) and reviewed & approved by pearu._ -- 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]
