Gabriel39 commented on PR #66227: URL: https://github.com/apache/doris/pull/66227#issuecomment-5864949164
Pushed 6009fdb795 to address the eight currently unresolved review comments. The third-party integration is also available separately against master in #68546. - Lossy bounded-string, timestamp, and decimal predicates/runtime filters stay residual until file/current schema equivalence can be established. No-key S3 DEFAULT configurations retain JNI credential-provider semantics. - Profiles now distinguish input, converted, and successfully applied Rust filters. The V2/schema-evolution differentials verify the actual Rust leaf, and the predicate suite adds positive/negative controls and historical-domain join cases. - Rust dependency builds recover missing/corrupt caches online, preserve offline behavior, and detect incomplete archive/identity publication for both libraries. Validation: 66 PaimonScanNode FE tests passed; all 40 tests in PaimonRustPredicateConverterTest and PaimonRustTableReaderTest passed in a focused ASAN build with shared test support; 8 build-helper tests passed. FE Checkstyle, clang-format 16, shell syntax, and Groovy parsing passed. The Paimon schema narrowing operations were smoke-tested. Both Rust libraries were built with Rust 1.97 and their installed identities verified. The temporary local test-source filter has been restored and is not included in the commit. The external Spark/Flink/Doris regression suites were not executed locally; their new end-to-end assertions still require CI validation. Existing upstream C API limitations (physical batch caps, late runtime-filter updates, cancellation during a blocking read, and physical I/O accounting) remain unchanged; this update does not claim to resolve those older deferred topics. -- 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]
