andygrove opened a new pull request, #4622:
URL: https://github.com/apache/datafusion-comet/pull/4622

   ## Which issue does this PR close?
   
   N/A. Documentation cleanup.
   
   ## Rationale for this change
   
   The Spark expression support reference (`expressions.md`) defines four 
statuses in its legend, but two of them, "⚠️ Incorrect by default" and "💤 Not 
currently planned", are not used by any row in the expression tables. Listing 
statuses that never appear is confusing and invites readers to look for rows 
that do not exist.
   
   The file-metadata functions (`input_file_name`, `input_file_block_start`, 
`input_file_block_length`) are technically Catalyst expressions, but their 
unsupported status is a scan-layer property and is already documented in the 
scan compatibility guide. Keeping a second copy in the expression reference 
risks the two pages drifting apart.
   
   ## What changes are included in this PR?
   
   - Remove the unused "Incorrect by default" and "Not currently planned" rows 
from the status legend.
   - Drop the intro sentence and "See also" wording that referenced the removed 
"Incorrect by default" status.
   - Replace the "File metadata" bullet in the not-planned list with a short 
cross-reference to the scan compatibility guide, which is the canonical place 
for that limitation.
   
   ## How are these changes tested?
   
   Documentation-only change; verified by rendering the page and confirming no 
remaining references to the removed statuses.
   


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

Reply via email to