Gabriel39 commented on PR #66227: URL: https://github.com/apache/doris/pull/66227#issuecomment-5868167797
Addressed the two newly confirmed compatibility findings in 17c6bbc3efa: - Historical FLOAT/DOUBLE to DECIMAL evolution now falls back to JNI, recursively through nested fields. Java's decimal-string rounding differs from Arrow's binary scaling: the persisted DOUBLE 1.005 case must read as DECIMAL 1.01, including the corresponding negative-value case. - All configured default/per-field aggregate names are checked against the pinned Rust function-name set before effective type checks. This covers unused defaults and key/sequence-field options that Java may ignore but Rust still validates. Supported defaults and explicit field overrides retain their existing precedence. Added routing and persisted Parquet regressions, including single-file unused collect defaults overridden by max under both aggregation and partial-update. Also made the nested-evolution regression fixture append-only so its positive Rust profile assertion tests schema reconciliation independently of the intentional multi-file primary-key fallback. Validation: both new routing tests and both persisted reproducers failed before the production fix. Afterward, all 89 FE JUnit tests passed (77 routing and 12 persisted-file tests), as did FE reactor Checkstyle, both changed Groovy syntax checks, and git diff --check. Full external SQL regression and native compilation were not run locally. Compilation and review have been requested through run buildall and /review. -- 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]
