nzw921rx commented on PR #10977: URL: https://github.com/apache/seatunnel/pull/10977#issuecomment-4598869867
@davidzollo Thank you for your review. I have made the following fixes based on the results, and welcome your review at any time - `isConstraintApplicable` now only checks the **head** option for required-ness — compareField in cross-field operators no longer forces applicability. This makes `optional(head) + required(compareField)` behave symmetrically: head absent → constraint skipped. - `MetadataExportCommand.exportCondition` now emits `conditionOperator` and `conditionOperatorCategory`, aligned with the REST API. - Fixed truncated Javadoc in `Conditions.java`; added supported operator inventory (17 operators, 4 categories). - `incompatible-changes.md` entry for `OptionValidationException` structured aggregation format. - Skipped — `Condition.*` factories are new code (never released), no compat entry needed. - `compareNumbers` no longer leaks raw values in error messages. - `OptionValidationException` gained `getRawMessage()`; `extractInnerMessage` replaced with direct accessor. - Added 3 new tests covering `optional(head) + required(compareField)` scenarios (head absent / both present / violation). -- 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]
