sahvx655-wq commented on PR #63910:
URL: https://github.com/apache/doris/pull/63910#issuecomment-4959150155

   The only check still red is BE UT (macOS), and it isn't this patch. From the 
job log, that run dies in the Check JAVA version step with "ERROR: The JAVA 
version is 25, it must be JDK-17" before any BE code is compiled or a single 
test executes, so nothing from this branch was actually exercised there. It's a 
stale failure from the June push; the run buildall only restarts the TeamCity 
pipelines, which is why that job never re-ran.
   
   On the substance everything came back green: Linux BE UT passes, the review 
at f64df2c found no blocking issues, and increment coverage on the new guards 
is 100% (13/13), covering both the output-capacity check and the 
large-block-length check from the earlier round. If you want a fresh macOS run 
I can push a rebase to retrigger it, otherwise I think this is ready for 
another look.


-- 
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]

Reply via email to