LuciferYang commented on issue #2837: URL: https://github.com/apache/iceberg-rust/issues/2837#issuecomment-5042187274
Thanks, that's really helpful context, and good to know the minimal fix already unblocks you. On the overlap: #2746 adds `_pos` as a virtual metadata column, which is a different name from the bare `pos` this fixes. They line up rather than collide. The guard here keys off the `_` prefix, so `_pos` stays a metadata column while a plain `pos` goes back to being a normal data column — if anything this keeps the two from being confused. I'll bring the merged `_pos` work into this branch so it sits on current main. Your `pos_col` remapping is exactly the kind of per-table special-casing this should let you drop, so thanks for laying out the use case. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
