gaogaotiantian commented on PR #57015: URL: https://github.com/apache/spark/pull/57015#issuecomment-4963285864
Overall I think it's nice to have less ruby code. But it seems like this is blocked by our checkout method. I also think that it's a bit weird to checkout code like we do now. We probably want the PR to be "up to date", but this does not guarantee anything. This only "kind of" guarantees that the CI of the PR is up to date when the CI is executed - we can still have stale result if we merge code after the CI executes. Customization in CI is sometimes not ideal. We can have a discussion about this any people might know why we have to do 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]
