friendlymatthew commented on code in PR #18253:
URL: https://github.com/apache/datafusion/pull/18253#discussion_r2465650262
##########
datafusion/datasource/src/file_scan_config.rs:
##########
@@ -175,9 +177,12 @@ pub struct FileScanConfig {
pub file_groups: Vec<FileGroup>,
/// Table constraints
pub constraints: Constraints,
- /// Columns on which to project the data. Indexes that are higher than the
- /// number of columns of `file_schema` refer to `table_partition_cols`.
- pub projection: Option<Vec<usize>>,
+ /// Physical expressions defining the projection to apply when reading
data.
+ ///
+ /// Each expression in the projection can reference columns from both the
file
+ /// schema and table partition columns. If `None`, all columns from the
table
+ /// schema are projected.
+ pub projection: Option<ProjectionExprs>,
Review Comment:
In that case, I'll rename the field to indicate it holds a list of
projections
--
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]
