XiaoHongbo-Hope commented on PR #659: URL: https://github.com/apache/paimon-rust/pull/659#issuecomment-5238874471
Thanks for the architectural review. Refactored in `bc852ec`: - removed the fixed-bucket sidecar from generic `TableWrite`; - split global statistics/planning, immutable plan, pure routing, and the dedicated planned writer; - kept no-plan local convenience as a separate wrapper around those primitives; - made builder overwrite mode apply to both writer and committer; - kept ownership validation only in `TableCommit` as a final backstop; integrations own selection and pre-write shuffling. Added planner-stat merging, planned-writer, overwrite, Rust, and C regressions. Workspace clippy and the CI-equivalent full-feature all-targets tests pass. -- 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]
