adriangb commented on PR #14362:
URL: https://github.com/apache/datafusion/pull/14362#issuecomment-2625987325
I see. That makes sense for something like row_id: once you write it to a
new file it may not be valid anymore. But for other metadata you might want to
round trip it as a system column. I think the argument could be made both ways.
But I do think there is a stronger argument for doing something fancier:
```sql
WITH cte AS (
SELECT id, _row_id FROM t
)
SELECT * FROM cte
```
IMO _row_id should be included.
So should we say that a system column stops being a system column once it's
projected?
--
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
