huan233usc opened a new pull request, #17855:
URL: https://github.com/apache/iceberg/pull/17855

   ## Summary
   
   Follow-up to #17451. Average value sizes were lost when content files 
crossed the planned REST scan JSON boundary, while `rewrite_table_path` 
retained a stale average for the position-delete `file_path` column after 
rewriting that path.
   
   - Preserve `avg-value-sizes` for both data and delete files in 
`ContentFileParser` JSON round trips and document the optional field in the 
REST OpenAPI schema.
   - Remove the `file_path` average value size when rewriting position-delete 
path bounds, preserve unrelated averages, and normalize an empty result to null.
   
   ## Test plan
   
   - [x] `./gradlew :iceberg-core:test --tests 
org.apache.iceberg.TestContentFileParser --tests 
org.apache.iceberg.TestFileScanTaskParser --tests 
org.apache.iceberg.TestDataTaskParser --tests 
org.apache.iceberg.TestRewriteTablePathUtil`
   - [x] `./gradlew :iceberg-core:spotlessCheck`
   - [x] Verified generated Python syntax and OpenAPI YAML syntax.
   
   The core tests used RoaringBitmap 1.6.19 temporarily because the configured 
Maven mirror returned 403 for 1.6.20; the dependency version is unchanged in 
this PR.
   
   ---
   **AI Disclosure**
   - Model: [unknown - human to fill in]
   - Platform/Tool: Cursor
   - Human Oversight: unreviewed
   - Prompt Summary: Implement and open a PR for the two follow-ups from 
#17451: REST content-file JSON round trips and rewrite-table-path average-size 
handling.


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