suxiaogang223 commented on PR #67395: URL: https://github.com/apache/doris/pull/67395#issuecomment-5730602616
Regarding the FFI backend placeholder: it is an intentional extension point for a future Rust FFI writer, not leftover code from the removed paimon-cpp reader. `branch-4.1` keeps the same backend boundary. The FE currently always sets `TPaimonWriteBackendType.JNI`, so this does not change current write behavior. If FFI is selected before its implementation is added, it fails explicitly with `NotSupported`. Keeping the Thrift enum and backend factory now lets the future implementation plug into the existing sink contract without reshaping that contract again. I therefore restored only this FFI extension point. `paimon_file_system_scheme_mappings` remains removed because it was specific to the old paimon-cpp reader and is unrelated to the future write backend. The FE compile failures were also fixed by adapting the recently merged snapshot row-count code and its tests to the Paimon 1.4.2 APIs (`totalRecordCount`, `Snapshot`, `FallbackReadFileStoreTable`, `TableQueryAuthResult`, and `CatalogEnvironment`). Verified locally with `./build.sh --fe`: **BUILD SUCCESS**. -- 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]
