Deprecate o.a.l.l.c.a.r.a.Duration class for removal

Author: ppkarwasz

log4j-core-test/src/test/java/org/apache/logging/log4j/core/appender/rolling/action/IfLastModifiedTest.java

@@ -23,6 +23,7 @@
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import java.nio.file.attribute.FileTime;
+import java.time.Duration;
 import java.util.List;
 import org.apache.logging.log4j.core.config.Node;
 import org.apache.logging.log4j.core.config.NullConfiguration;
@@ -43,12 +44,6 @@
 @SetSystemProperty(key = "log4j2.status.entries", value = "10")
 class IfLastModifiedTest {
 
-    @Test
-    public void testGetDurationReturnsConstructorValue() {
-        final IfLastModified filter = IfLastModified.createAgeCondition(Duration.parse("P7D"));
-        assertEquals(0, filter.getAge().compareTo(Duration.parse("P7D")));
-    }
-
     @Test
     public void testAcceptsIfFileAgeEqualToDuration() {
         final IfLastModified filter = IfLastModified.createAgeCondition(Duration.parse("PT33S"));

log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/action/Duration.java

@@ -30,7 +30,9 @@
  * This implementation does not support fractions or negative values.
  *
  * @see #parse(CharSequence)
+ * @deprecated since 2.24.0 use {@link java.time.Duration} instead.
  */
+@Deprecated
 public class Duration implements Serializable, Comparable<Duration> {
     private static final long serialVersionUID = -3756810052716342061L;
 
@@ -66,6 +68,13 @@ public class Duration implements Serializable, Comparable<Duration> {
     private static final Pattern PATTERN = Pattern.compile(
             "P?(?:([0-9]+)D)?" + "(T?(?:([0-9]+)H)?(?:([0-9]+)M)?(?:([0-9]+)?S)?)?", Pattern.CASE_INSENSITIVE);
 
+    /**
+     * @since 2.24.0
+     */
+    public static Duration ofMillis(final long millis) {
+        return new Duration(millis / 1000L);
+    }
+
     /**
      * The number of seconds in the duration.
      */

log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/action/IfLastModified.java

@@ -19,6 +19,7 @@
 import java.nio.file.Path;
 import java.nio.file.attribute.BasicFileAttributes;
 import java.nio.file.attribute.FileTime;
+import java.time.Duration;
 import java.util.Arrays;
 import java.util.Collections;
 import java.util.List;
@@ -50,8 +51,12 @@ private IfLastModified(final Duration age, final PathCondition[] nestedCondition
         this.nestedConditions = PathCondition.copy(nestedConditions);
     }
 
-    public Duration getAge() {
-        return age;
+    /**
+     * @deprecated since 2.24.0 without a replacement.
+     */
+    @Deprecated
+    public org.apache.logging.log4j.core.appender.rolling.action.Duration getAge() {
+        return org.apache.logging.log4j.core.appender.rolling.action.Duration.ofMillis(age.toMillis());
     }
 
     public List<PathCondition> getNestedConditions() {
@@ -89,20 +94,29 @@ public void beforeFileTreeWalk() {
         IfAll.beforeFileTreeWalk(nestedConditions);
     }
 
+    /**
+     * @deprecated since 2.24.0 use {@link #createAgeCondition(Duration, PathCondition...)} instead.
+     */
+    @Deprecated
+    public static IfLastModified createAgeCondition(
+            final org.apache.logging.log4j.core.appender.rolling.action.Duration age,
+            final PathCondition... pathConditions) {
+        return createAgeCondition(Duration.ofMillis(age.toMillis()), pathConditions);
+    }
+
     /**
      * Create an IfLastModified condition.
      *
      * @param age The path age that is accepted by this condition. Must be a valid Duration.
-     * @param nestedConditions nested conditions to evaluate if this condition accepts a path
+     * @param pathConditions Nested conditions to evaluate if this condition accepts a path.
      * @return An IfLastModified condition.
+     * @since 2.24.0
      */
     @PluginFactory
     public static IfLastModified createAgeCondition(
-            // @formatter:off
             @PluginAttribute("age") @Required(message = "No age provided for IfLastModified") final Duration age,
-            @PluginElement("PathConditions") final PathCondition... nestedConditions) {
-        // @formatter:on
-        return new IfLastModified(age, nestedConditions);
+            @PluginElement("pathConditions") final PathCondition... pathConditions) {
+        return new IfLastModified(age, Objects.requireNonNull(pathConditions));
     }
 
     @Override

log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/action/package-info.java

@@ -18,7 +18,7 @@
  * Support classes for the Rolling File Appender.
  */
 @Export
-@Version("2.20.2")
+@Version("2.24.0")
 package org.apache.logging.log4j.core.appender.rolling.action;
 
 import org.osgi.annotation.bundle.Export;

log4j-core/src/main/java/org/apache/logging/log4j/core/config/plugins/convert/TypeConverters.java

@@ -32,11 +32,11 @@
 import java.nio.file.Paths;
 import java.security.Provider;
 import java.security.Security;
+import java.time.Duration;
 import java.util.UUID;
 import java.util.regex.Pattern;
 import org.apache.logging.log4j.Level;
 import org.apache.logging.log4j.Logger;
-import org.apache.logging.log4j.core.appender.rolling.action.Duration;
 import org.apache.logging.log4j.core.config.plugins.Plugin;
 import org.apache.logging.log4j.core.util.CronExpression;
 import org.apache.logging.log4j.status.StatusLogger;
@@ -224,11 +224,26 @@ public Double convert(final String s) {
     }
 
     /**
-     * Converts a {@link String} into a {@link Duration}.
+     * Converts a {@link String} into a {@link org.apache.logging.log4j.core.appender.rolling.action.Duration}.
      * @since 2.5
+     * @deprecated since 2.24.0 use {@link JavaDurationConverter}
      */
     @Plugin(name = "Duration", category = CATEGORY)
-    public static class DurationConverter implements TypeConverter<Duration> {
+    @Deprecated
+    public static class DurationConverter
+            implements TypeConverter<org.apache.logging.log4j.core.appender.rolling.action.Duration> {
+        @Override
+        public org.apache.logging.log4j.core.appender.rolling.action.Duration convert(final String s) {
+            return org.apache.logging.log4j.core.appender.rolling.action.Duration.parse(s);
+        }
+    }
+
+    /**
+     * Converts a {@link String} into a {@link Duration}.
+     * @since 2.24.0
+     */
+    @Plugin(name = "JavaDuration", category = CATEGORY)
+    public static class JavaDurationConverter implements TypeConverter<Duration> {
         @Override
         public Duration convert(final String s) {
             return Duration.parse(s);

log4j-core/src/main/java/org/apache/logging/log4j/core/config/plugins/convert/package-info.java

@@ -20,7 +20,7 @@
  * attributes in plugin factory methods.
  */
 @Export
-@Version("2.20.2")
+@Version("2.24.0")
 package org.apache.logging.log4j.core.config.plugins.convert;
 
 import org.osgi.annotation.bundle.Export;

src/changelog/.2.x.x/deprecate_duration.xml

@@ -0,0 +1,7 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<entry xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+       xmlns="http://logging.apache.org/log4j/changelog"
+       xsi:schemaLocation="http://logging.apache.org/log4j/changelog https://logging.apache.org/log4j/changelog-0.1.3.xsd"
+       type="changed">
+  <description format="asciidoc">Deprecate `org.apache.logging.log4j.core.appender.rolling.action.Duration` class for removal.</description>
+</entry>