nastra commented on PR #13700: URL: https://github.com/apache/iceberg/pull/13700#issuecomment-3143314950
> @nastra: In #3029 you were working on the Fixed type, but abandoned it, because > > > TBH I'm uncertain the approach is correct or not because I'm not sure if this statement is still correct (https://github.com/apache/iceberg/pull/3029/files#diff-80bc724de9a4dd358c4544fcf00e00139145c6763a5d0280e0bd0793a0fd4003L366-L368) > > OTOH, the Fixed type seems to be supported by the Arrow reader. Do you happen to remember a bit more about the history here? Back then I wasn't sure whether we could just remove `FixedWidthBinaryPageReader` from the codebase. Over time we did actually remove it and so I think we should be able to fully support `FIXED` type -- 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]
