[PR] feat(datafusion): support the $aggregation_fields system table [paimon-rust]

["jackylee-ch (via GitHub)" <gi...@apache.org>]

jackylee-ch opened a new pull request, #937:
URL: https://github.com/apache/paimon-rust/pull/937

   Java exposes `AggregationFieldsTable` (`<table>$aggregation_fields`) so a user can see which aggregate function each column of an aggregation merge-engine table is configured with. The DataFusion integration had no equivalent.
   
   This adds the `$aggregation_fields` provider: for every schema field it emits `field_name`, `field_type` (the SQL type string), `function` and `function_options` — the values and keys of that field's `fields.<name>.*` options, rendered the way Java's multimap does (`[sum]`, `[fields.total.aggregate-function]`, or `[]`) — and `comment`. Entries are sorted by option key so the output is stable.
   
   Tested by creating an aggregation table and asserting a configured column renders `[sum]` while an unconfigured one renders `[]`.
   


-- 
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: issues-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] feat(datafusion): support the $aggregation_fields system table [paimon-rust]

["JingsongLi (via GitHub)" <gi...@apache.org>]

JingsongLi commented on PR #937:
URL: https://github.com/apache/paimon-rust/pull/937#issuecomment-5831662716

   Requirement fit: SUPPORTED. `$aggregation_fields` gives SQL users a useful view of aggregation column configuration. I checked the provider registration, branch/system-table routing, schema, field-option extraction, and output against Paimon Java `AggregationFieldsTable`; no actionable correctness issue found at 590da395. The DataFusion end-to-end SQL test passes, `cargo fmt --all -- --check` and `git diff --check` pass, the branch merges cleanly with current main, and all 14 CI checks are green. Small cleanup: remove the leftover `// MARKER_AGG_FIELDS` comment from the new test before merging.


-- 
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: issues-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] feat(datafusion): support the $aggregation_fields system table [paimon-rust]

["JingsongLi (via GitHub)" <gi...@apache.org>]

JingsongLi merged PR #937:
URL: https://github.com/apache/paimon-rust/pull/937


-- 
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: issues-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org