github-actions[bot] commented on code in PR #66546: URL: https://github.com/apache/doris/pull/66546#discussion_r3868576381
########## thirdparty/arrow-paimon-vars.sh: ########## @@ -21,12 +21,24 @@ # Keep the Arrow/Paimon source closure in a dedicated file so targeted CI can # distinguish this stack from unrelated thirdparty changes. -# arrow +# Arrow 24 is installed in a versioned prefix. The unversioned install prefix is Review Comment: [P1] Keep the current branch-4.1 consumer compatible with this layout The live `branch-4.1` already contains the Arrow 24 backport and still validates/imports Arrow/Paimon 24 from the unversioned `installed` root (including `libarrow_compute.a`); its BE-UT script refreshes the official install-only tree from this same shared automation asset. This change replaces that root with the Arrow/Paimon 17 contract and puts 24 only under `installed/arrow-24.0.0`. Once the asset uses this layout, the current branch rejects it and cannot recover, while a source checkout rebuilds 24 into the root and removes compatibility for the pre-upgrade revisions this change is meant to preserve. Please coordinate/land a branch-4.1 consumer change that selects the versioned 24 prefix before publishing this artifact, and cover that branch's validator/import contract in the lifecycle test. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
