Gabriel39 commented on PR #67904: URL: https://github.com/apache/doris/pull/67904#issuecomment-5652357397
Addressed the five review findings in 75ceac12b8: - Preserve bound Paimon schema options and replay only catalog/runtime overrides; added a bucket/sequence-key warm-cache rename test. - Restore the main fallback leaf independently, retain the fallback schema/branch, and preserve the privilege checker; verified fallback-only rows and SELECT denial enforcement. - Leave TIMESTAMP(7..9) comparisons residual; covered nanosecond values across comparison operators. - Pin Iceberg partition spec IDs with schema IDs and resolve source names against the pinned schema; covered a shared latest-cache hit after live rename/spec evolution. - Converted all five fixed result checks to order_qt_ cases, generated the .out with the regression harness, and reran successfully. Validation: 1,963 connector tests passed and 6 skipped; one pre-existing Iceberg MERGE test failure was reproduced on the previous PR head and excluded. FE Checkstyle, FE build, and plugin packages passed. Local regression used the current FE/JNI extension with an existing BE and disabled the FE local-shuffle planner only on that isolated test instance for compatibility. The PR description has been updated. GitHub review-reply and thread-resolution APIs are returning server errors, so this top-level comment records the fixes while those discussion updates remain unavailable. -- 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]
