adriangb commented on PR #14362:
URL: https://github.com/apache/datafusion/pull/14362#issuecomment-2622449713

   I don't know if there's any other "known" metadata, but I feel like it would 
be good to have an extension trait along the lines of:
   
   ```rust
   /// Extension of [`Field`] to manage DataFusion specific field metadata.
   pub trait DataFusionFieldInfo {
       /// A nice docstring!
       fn is_system_column(&self) -> bool;
      /// Another nice docstring :)
       fn as_system_column(mut self) -> Self;
   }
   
   impl DataFusionFieldInfo for Field { ... }
   ```
   
   And then we can call `field.is_system_column()` from anywhere that needs to 
check.
   


-- 
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