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

Reply via email to