ctubbsii commented on issue #1117: URL: https://github.com/apache/accumulo/issues/1117#issuecomment-872582136
The original ticket did suggest using the enum constructor parameter to store this information, but annotations might be easier. It'd just be annoying to have a new annotation for *every* enum, since that's a lot of new lines to add to the file. It might be best to store "since" information in the enum, passed as a constructor parameter, and only use the annotations for `@Deprecated` and `@ReplacedBy`, since those are rarer. Even if this doesn't save space, it can be used to guarantee that everything gets "since" information, because parameters can be made mandatory, and it's easy to accidentally omit an annotation. Though, a unit test could be used to ensure everything's annotated with "since" information if you go that route. -- 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]
