wjixiang commented on issue #2837: URL: https://github.com/apache/iceberg-rust/issues/2837#issuecomment-5037058776
Hi @LuciferYang, Thanks so much for the detailed breakdown and for surfacing both the minimal fix and the deeper cleanup option — really appreciate the thoroughness. As someone who's just using Iceberg and not deeply familiar with the internals, I don't feel qualified to weigh in strongly on which approach is technically better. That said, from a user perspective, your current minimal fix would already be a huge help — it would unblock my use case completely. For some context: I'm building a query engine on top of Datafusion for parsing genomic data, and the pos column collision is exactly what I'm running into. My current workaround is to remap pos to pos_col at the query layer, which works but feels brittle and forces me to special-case Iceberg tables. One thing I noticed — there's parallel work going on in #2607/#2746 to add _pos as an official metadata column. I'm not sure how that intersects with your PR, but I figured it's worth mentioning in case it influences your approach. Either way, I'm happy to go with whatever you and the maintainers think is best. Just wanted to share my use case and say that I'd really appreciate having this fix landed. Thanks again for picking this up! -- 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]
