Re: [PR] AVRO-4050: Deprecate MapUtil computeIfAbsent [avro]

["martin-g (via GitHub)" <gi...@apache.org>]

martin-g commented on code in PR #3142:
URL: https://github.com/apache/avro/pull/3142#discussion_r1749875701


##########
lang/java/avro/src/main/java/org/apache/avro/util/MapUtil.java:
##########
@@ -27,13 +27,13 @@ private MapUtil() {
   }
 
   /**
-   * A temporary workaround for Java 8 specific performance issue JDK-8161372
-   * .<br>
-   * This class should be removed once we drop Java 8 support.
+   * A temporary workaround for Java 8 specific performance issue JDK-8161372.
    *
    * @see <a href=
    *      "https://bugs.openjdk.java.net/browse/JDK-8161372">JDK-8161372</a>
+   * @deprecated As of JDK 1.9 this issue has been resolved.
    */
+  @Deprecated

Review Comment:
   ```suggestion
     // TODO: Remove for 1.13.0 or later
     @Deprecated
   ```



-- 
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: issues-unsubscribe@avro.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org