JingsongLi commented on PR #615:
URL: https://github.com/apache/paimon-rust/pull/615#issuecomment-5162046894
- `Schema::validate_sequence_field` omits the `options.fieldAggFunc(field)
== null` validation found in Java’s `SchemaValidation.validateSequenceField`.
- Currently, Rust incorrectly accepts:
`sequence.field=ts`
`fields.ts.aggregate-function=sum`
- I added a temporary regression test; it was expected to return an error
from `build()`, but it actually returned `Ok(Schema)`. This issue has been
consistently reproducible.
- This is not only incompatible with Java; during a partial update, the
sequence field may be skipped by the aggregation configuration logic,
potentially leading to a situation where “the configuration is written to the
metadata but is silently ignored in practice.”
- I recommend that `validate_sequence_field` directly reject all
`fields.<sequence-field>.aggregate-function` entries, and that create/alter
tests be added.
--
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]
