kennknowles commented on PR #35150: URL: https://github.com/apache/beam/pull/35150#issuecomment-3081190993
I think that the many cases of `@NonNull` and `@Nullable` applied to type variables indicates a problem. Each one really needs a comment justifying it, because it is unusual and actually a somewhat inflexible situation. Probably you can remove most of them. And some places where you have a type variable and you need to constrain what types it can be instantiated with, you would use `extends @NonNull Object` (the default if you do not annotate is `extends @Nullable Object` which allows the type variable to be instantiated with any type) -- 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
