wwbmmm commented on PR #3486: URL: https://github.com/apache/brpc/pull/3486#issuecomment-5425526568
> The unittest ci takes 20 minutes to execute, so merging other build ci (e.g., protobuf build ci) into one does not affect the overall execution time. I think we can just remove some param combinations to keep the job num <= 20. Merging jobs into one may increase failure rate. BTW, I think the unit test takes 20 minutes is not ideal. Maybe we can optimize those slow cases in the future. -- 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]
