lasdf1234 commented on PR #12942: URL: https://github.com/apache/gravitino/pull/12942#issuecomment-5695143469
> Added all three points: > > 1. Section 5.2 now defines the trigger expression and configurable defaults: count >= 500, or count >= 100 with average size < 8 MiB (8,388,608 bytes), including boundary examples. > 2. Sections 5.3 and 5.4 resolve the target spec once, collect only that spec, and store `custom-manifest-number-by-spec` and `custom-avg-manifest-size-by-spec` as two object-valued rows keyed by spec ID. Collection merges only the resolved entry and preserves other specs. Evaluation and the submitted job use that exact resolved ID, including when the original request omitted it. > 3. Section 9 commits the initial runtime/CI target to Spark 3.5 and Iceberg 1.11.0. > > Updated the PR plan to cover threshold boundaries, map updates, and default-spec changes between collection and submission. The former open questions are now recorded as resolved decisions. These are design changes; policy/collector implementation remains in the follow-up PR plan. Yes all issue i mentioned that you have fixed. -- 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]
