chenkovsky commented on PR #14057: URL: https://github.com/apache/datafusion/pull/14057#issuecomment-2621372943
I also have an another approach. ```rust struct ColumnIndex { pub index: usize, pub is_metadata_column: bool } impl Into<usize> for ColumnIndex { } impl From<usize> for ColumnIndex { } ``` then we can hide METADATA_OFFSET logic without performance concern. what do you think about this? -- 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...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org