Gabriel39 commented on PR #67784: URL: https://github.com/apache/doris/pull/67784#issuecomment-5673295086
Pushed acd7d63c0d with the review fix and follow-ups for regression builds 1045934 and 1045935: - Preserve passive timestamp/VARIANT columns rejected by the lazy-materialization probe in both file and TVF phase-one scan outputs. This addresses the shared missing-child-slot failures. - Stop Iceberg schema-requirement traversal at VARIANT carriers; document keys are not schema field IDs. - Correct the profile assertion to use logical SelectedRows, bind scanner options directly to the JNI compatibility assertions, and align binary/statistics/CTAS/manifest and isolated round-robin fixture expectations with their actual contracts. Validation: FE compilation; 136 unit tests passing; FE Checkstyle; Groovy syntax checks for all 8 changed suites; clang-format 16 checks for all 85 C++ files affected by the PR. Binary golden changes were checked for lossless byte conversion. Full integration results remain pending the requested buildall rerun. Three muted failures are not claimed fixed: test_hdfs_parquet_group0 numeric rendering, test_paimon_write_thread_lifecycle JVM thread baseline, and test_read_cluster_var_property cache-counter assertion. Their assertions have not been relaxed; further runtime evidence is needed to establish their causes. -- 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]
