srielau commented on PR #58317: URL: https://github.com/apache/spark/pull/58317#issuecomment-5511322516
Addressed in 57a9c672d18. `TreeNodeTag` is excluded from canonicalization/`sameResult`, so a preserve-only cached ORC scan that truncated `"abcdef"` to `"abcd"` could replace a standard-semantics scan and skip `EXCEED_LIMIT_LENGTH`. The bound mode is now case-class state on `LogicalRelation`, `DataSourceV2Relation`, `HiveTableRelation`, and `FileSourceScanExec`; `OrcScan.equals` compares it as well. Added V1/V2 coverage that preserve-only vs standard plans are not `sameResult` and that `CACHE TABLE` under preserve-only is not reused under standard semantics. -- 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]
