Yicong-Huang commented on PR #55677: URL: https://github.com/apache/spark/pull/55677#issuecomment-4392035891
> The CI is intentionally enabled only on PRs (not main commits). This is supposed to be a warning like - you need to fix the client ASAP. I think the proper protocol is to have both PRs ready, and merge them sequentially in a short window. I think we should note this as a warning somehow. for example post a comment. because some reviewers/committers expect a green CI before review: If a PR's CI is failing, they may not spend time to review it. -- 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]
