Gabriel39 commented on PR #67784: URL: https://github.com/apache/doris/pull/67784#issuecomment-5690682161
Follow-up 94969b5299 addresses the four review comments and the reported CI failures. - P0: the binary CTAS audit entry matches the BE abort in RawValue::zlib_crc32. The subsequent backend-unavailable failures were cascading failures. The image-check alarm followed the failed stop of the already-crashed BE, which skipped FE shutdown; FE logs show successful image saves, not a checkpoint serialization exception. - Cloud P0: added VARBINARY to Column.toPb field-length/index-length handling, with bounded, unbounded and nested metadata tests. - FE UT: corrected the stale trailing-zero equality expectation and added the reverse ordering assertion. Local verification before pushing: Release BE build and 100 BE tests; FE package build and 76 FE tests; test_varbinary_sql_support and test_varbinary_native_paths (2 suites, 0 failures); FE Checkstyle and clang-format 16 checks. The Cloud-specific failure was reproduced and verified through the actual Column.toPb unit-test path; a full Cloud runtime suite was not run locally. Full CI and a new review have been requested. -- 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]
