jackylee-ch opened a new pull request, #638: URL: https://github.com/apache/paimon-rust/pull/638
Java `SchemaValidation#validateSequenceField` checks `fieldAggFunc(field) == null` for every field in `sequence.field`, and that validator runs for every merge engine. The Rust equivalent lived inside `AggregationConfig::validate_field_scoped_options`, which is only reached when `merge-engine=aggregation` — so on the default deduplicate engine a schema combining `sequence.field = 'ts'` with `fields.ts.aggregate-function = 'sum'` was silently accepted and persisted. **Fix**: hoist the check into a free `validate_no_aggregation_on_sequence_field` keyed only on the option map, and call it from `Schema::new` and `TableSchema::apply_changes` alongside the other sequence-field validations, so it no longer depends on the merge engine. The existing aggregation-engine test is widened to cover all five engines, plus create/alter tests on the default engine. -- 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]
