ctubbsii commented on code in PR #3090:
URL: https://github.com/apache/accumulo/pull/3090#discussion_r1030011372
##########
core/src/main/java/org/apache/accumulo/core/data/Mutation.java:
##########
@@ -1507,6 +1507,7 @@ public int size() {
* the peer to add
* @since 1.7.0
*/
+ @Deprecated(since = "2.1.1")
Review Comment:
@keith-turner suggested
> ```java
> * @deprecated since 2.1.1 This should have been deprecated in 2.1.0 in
order to strictly follow semver. However it was missed and it must be removed
in 3.0.0 if all of replication is removed. Therefore the deprecation was added
in 2.1.1 which is a bit out of line with semver.
> */
> @Deprecated(since = "2.1.1")
> ```
Yes, we can add that in the javadoc, as well as the release notes. I'll take
a look at how the annotation 'since' renders with the javadoc description to
see if I can make them mesh well together without being redundant.
--
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]
