Gabriel39 commented on PR #67784: URL: https://github.com/apache/doris/pull/67784#issuecomment-5707306983
Rebased onto branch-4.1 (`3a33142dc3`) and resolved the CatalogMgr conflict while preserving both the upstream Lance admission guards and this PR's journaled catalog migration. Follow-up fix: `be1962dfb6`. - Do not format the nested payload of NULL timestamps during string CAST. This fixes the date-functions failure without relaxing validation of non-NULL invalid timestamps. - Convert Remote Doris metadata timeout seconds to Thrift milliseconds, with zero/overflow handling and tests covering both metadata RPC paths. - Align the MySQL upgrade expectations with TIMESTAMPTZ instant semantics, verified against a real MySQL 5.7 UTC fixture. - Make historical timezone output tests explicit about Asia/Shanghai and preserve its second-resolution offset. Local verification before pushing: - FE package and Checkstyle passed; 125 targeted FE tests passed. - BE production/unit builds passed; 382 BE tests passed. - All 119 affected C++ files passed clang-format 16. - Freshly built FE/BE passed all five original non-muted failing suites plus the new NULL-timestamp regression: 6 passed, 0 failed, 0 fatal, 0 skipped. The upgrade suite exercised the affected JDBC MySQL branch. The existing muted Lance discriminator and schema-change failures are not included in the fixed-case count. The Lance failure also occurs on unrelated successful PR builds. Full CI has been requested again; these local results do not assert a complete CI pass. -- 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]
