Gabriel39 commented on PR #67784: URL: https://github.com/apache/doris/pull/67784#issuecomment-5699014385
Rebased onto branch-4.1 at 8bbe4c5fad; current head is 90a55f0efc. Conflict resolution preserves the upstream InvalidArgument classification needed by TRY_CAST and the UTC-only diagnostics that avoid secondary timestamp formatting failures. Both the upstream MaxCompute ZSTD round-trip test and the timestamp/DST tests are retained. Local validation also exposed duplicate Surefire argLine entries introduced by the dependency update. The follow-up commit combines the Arrow JVM access options with the JMockit and JaCoCo agents; the previously failing ZSTD test now passes without weakening any assertions. Validation: 260 FE tests and 13 MaxCompute tests passed with no failures, errors, or skips. The BE build and 375 focused tests passed; BE sources and dependencies are unchanged by the final FE-only rebase. FE Checkstyle and clang-format 16 checks for all 117 affected C++ files passed. Range-diff confirms the existing PR changes are preserved. GitHub now reports the PR as mergeable. Build and review have been requested again; remote CI results are pending. -- 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]
