szehon-ho commented on PR #58300: URL: https://github.com/apache/spark/pull/58300#issuecomment-5419233480
cc @zhengruifeng -- this follows on from the discussion in #58165. Both of these sit on the empty-inference path, which that PR made newly reachable from the normal merge flow, so you have the most context on whether the prompt behaviour here matches what you want for your cases 2 and 3. One open question I left out deliberately: the resolution gate added in #58165 now allows only `Fixed`. SPARK also resolved 29 tickets as `Done` and 5 as `Implemented` over the last year, against 4686 as `Fixed`. Those are hand-set resolutions that still mean the work landed, and a backport merge on one is now skipped silently where it used to prompt. Happy to widen the allowlist in a separate PR if you think it is worth it, or leave `Fixed`-only as the conservative choice. cc @Yicong-Huang as you weighed in on the resolved-ticket handling there too. -- 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]
