kevinjqliu commented on PR #16946: URL: https://github.com/apache/iceberg/pull/16946#issuecomment-5498192817
> My main point is that with paths-ignore, it's getting harder to maintain, since we need to update it when a Spark version is added or removed. agreed! i need to revive that PR 😄 I think the main point is that these workflows are failed open: we can bypass other spark versions IFF the PR contains only changes to that 1 specific spark version. otherwise, we run all. So we treat it as an optimization and still preserve correctness -- 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]
