FANNG1 commented on PR #10189: URL: https://github.com/apache/gravitino/pull/10189#issuecomment-4015485441
@jerryshao Thanks for the review. For the `properties()` vs `rules()` question: yes, this is intentional. - `properties()` keeps stable strategy identity/config selection fields (`strategy.type`, `job.template-name`). - `rules()` carries runtime evaluation knobs (thresholds, weights, trigger/score expressions, and `job.options.*`). I also added an inline code comment in `IcebergDataCompactionContent.properties()` to make this boundary explicit. The fix is in local commit `3b9d078d1` and I’ll push it to the PR branch after confirmation. -- 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]
