zqr10159 commented on PR #4294: URL: https://github.com/apache/hertzbeat/pull/4294#issuecomment-5137531180
The remaining review blockers are addressed in the latest head (`7768405ced`). The PR now targets current `master` directly and contains exactly one commit, so it no longer depends on #4263. Java plus all shipped YAML defaults are `required`; `optional` remains available only through an explicit rollout override. The new default regression failed on the prior head (`expected REQUIRED but was OPTIONAL`) and now passes. Latest local validation: 21 remoting authentication/compatibility tests, 16 targeted manager/collector tests, and the 24-module startup source package proof all passed. Full GitHub CI has now been triggered by the master retarget/push. -- 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]
