JingsongLi commented on PR #949: URL: https://github.com/apache/paimon-rust/pull/949#issuecomment-5831358640
Requirement fit: **SUPPORTED**. Implementation: **CLEAN** at `4552b103`. A shared reservation budget was already supported in the C layer but unreachable from Go. I traced context creation, ABI structs, metric sampling, and attachment through the read, write, and postpone fixed-bucket builders into the Rust resource clones. Closing the Go context after attachment leaves the builder-owned clone valid; the new lock prevents its handle from being freed during an attach or metric call. I found no actionable issue in these paths. Local verification: built the exact-head `paimon-c` library, then ran the Go resource tests against it. `TestResourceContextMetricsAndClose` and `TestReadBuilderWithResourceBudget` passed. The two write tests skipped locally because their Docker-generated warehouse tables are unavailable. The PR CI completed successfully, including `integration (go)` and all 14 jobs. `git diff --check main...HEAD` passed. -- 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]
