damccorm commented on PR #17429: URL: https://github.com/apache/beam/pull/17429#issuecomment-1108594304
> but how do you feel about merging this after the 2.39 cut, so the first release that requires Go 1.18 is 2.40? Another option (which I might like even more) is to review this PR as is, and then undo all the changes except for the template file and specialize/main.go. That would allow me to keep putting up my remaining generics changes for review, so that when we reach 2.40 all that needs to happen is running `go generate`, go.mod and CI changes, and adding back `registration_tests.go`. That would save me some hard context shifts, but it may be more trouble than its worth - regardless, I'd prefer we prioritize finishing the [watermark reviews](https://github.com/apache/beam/pull/17374), and by the time those are all through 2.39 may have been cut (or be close to being cut) -- 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: github-unsubscr...@beam.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
