ding-young commented on PR #7627: URL: https://github.com/apache/arrow-rs/pull/7627#issuecomment-3159989820
Hi @LiaCastaneda :) I just ran the reproducer test, and it passes without error on the fix-row-converter branch. However, this PR takes the approach of correcting the data type(i.e. flattened type https://github.com/apache/arrow-rs/issues/7169#issuecomment-2675211398), rather than preserving the original dictionary encoding. I can easily follow up on the minor review comment about adding a test, but currently I don’t have time to rewrite the decoding logic to preserve the original encoding. What kind of solution were you thinking of? If your main goal is just to suppress the warning, then I can finish up this PR within a day and ping you again. But if you’re planning to work on a solution that preserves the original dictionary encoding, feel free to take it over! Let me know what you think :) -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
