u70b3 commented on PR #67305: URL: https://github.com/apache/doris/pull/67305#issuecomment-5582552322
Closing as obsolete — the underlying problem no longer exists on master. This PR kept the pre-#67158 semantic marker `cb82e41ba…` accepted in `PAIMON_LEGACY_BUILD_FINGERPRINTS` so the Paimon prebuilt shipped in the compile image would keep passing validation. Since this was opened, #67385 (`[chore](paimon) remove paimon-cpp dependency`, merged 2026-09-03) removed paimon-cpp from thirdparty entirely, including: - `thirdparty/arrow-paimon-vars.sh` — the file this PR modifies (hence the modify/delete conflict GitHub reports) - the whole prebuilt fingerprint validation machinery (`PAIMON_LEGACY_BUILD_FINGERPRINTS`, `paimon_prebuilt_valid`, etc.) - the paimon-cpp patches and the arrow-paimon lifecycle test There is no Paimon prebuilt validation left on master that could reject the in-circulation marker; Paimon reads now go through JNI and native readers. The CI failure this PR addressed (`Paimon build fingerprint does not match selected inputs`) cannot occur anymore, so no fix is needed. -- 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]
