Rebased and update after PR #454 merge

Author: Abigael-JT

src/main/java/com/teragrep/pth10/ast/DefaultTimeFormat.java

@@ -47,6 +47,7 @@
 
 import java.text.ParseException;
 import java.text.SimpleDateFormat;
+import java.util.Arrays;
 import java.util.Date;
 
 /**
@@ -97,7 +98,10 @@ public Date parse(String time) {
             catch (ParseException ignored) {
             }
         }
-        throw new RuntimeException("Check that the timestamp or the relative time value is in the correct format (Supported timestamp formats <>)");
+        throw new RuntimeException(
+                "Check that the timestamp or the relative time value is in the correct format (Supported timestamp formats: "
+                        + Arrays.toString(formats) + ")"
+        );
     }
 
     private Date parseDate(String time, String timeFormat) throws ParseException {

src/main/java/com/teragrep/pth10/ast/time/InstantTimestamp.java

@@ -59,7 +59,7 @@
 
 public final class InstantTimestamp implements DPLTimestamp {
 
-    private static final Logger LOGGER = LoggerFactory.getLogger(EpochTimestamp.class);
+    private static final Logger LOGGER = LoggerFactory.getLogger(InstantTimestamp.class);
     private final String value;
     private final String timeformat;
 
@@ -76,15 +76,19 @@ public Instant instant() {
         }
         catch (NumberFormatException ne) {
             LOGGER.debug("Could not parse relative timestamp, trying default formats");
-            rv = instantFromString(value, timeformat);
+            rv = instantFromString(value, timeformat, ne);
         }
 
         return rv;
     }
 
     // Uses defaultTimeFormat if timeformat is null and DPLTimeFormat if timeformat isn't null (which means that the
     // timeformat= option was used).
-    private Instant instantFromString(final String value, final String timeFormatString, final NumberFormatException cause) {
+    private Instant instantFromString(
+            final String value,
+            final String timeFormatString,
+            final NumberFormatException cause
+    ) {
         final String unquotedValue = new UnquotedText(new TextString(value)).read(); // erase the possible outer quotes
         final Instant timevalue;
         if (timeFormatString == null || timeFormatString.isEmpty()) {

src/test/java/com/teragrep/pth10/DefaultTimeFormatTest.java

@@ -123,6 +123,9 @@ void invalidTimeformat() {
                 .assertThrows(RuntimeException.class, () -> new DefaultTimeFormat().getEpoch(time));
 
         Assertions
-                .assertEquals("TimeQualifier conversion error: <12/34/2020:10:25:40> can't be parsed.", rte.getMessage());
+                .assertEquals(
+                        "Check that the timestamp or the relative time value is in the correct format (Supported timestamp formats: [MM/dd/yyyy:HH:mm:ss, yyyy-MM-dd'T'HH:mm:ss.SSSXXX, yyyy-MM-dd'T'HH:mm:ss.SSS, yyyy-MM-dd'T'HH:mm:ssXXX, yyyy-MM-dd'T'HH:mm:ss])",
+                        rte.getMessage()
+                );
     }
 }

src/test/java/com/teragrep/pth10/EarliestLatestTest.java

@@ -359,7 +359,7 @@ public void defaultFormatInvalidInputTest() { // MM/dd/yyyy:HH:mm:ss 2013-07-15
                 });
         Assertions
                 .assertEquals(
-                        "Error parsing <31/31/2014:00:00:00>. Check that the timestamp or the relative time value is in the correct format (Supported timestamp formats <MM/dd/yyyy:HH:mm:ss>, <yyyy-MM-dd'T'HH:mm:ss>, <yyyy-MM-dd'T'HH:mm:ssXXX>). Unknown relative time modifier string [31/31/2014:00:00:00].",
+                        "Error parsing <31/31/2014:00:00:00>. Check that the timestamp or the relative time value is in the correct format (Supported timestamp formats: [MM/dd/yyyy:HH:mm:ss, yyyy-MM-dd'T'HH:mm:ss.SSSXXX, yyyy-MM-dd'T'HH:mm:ss.SSS, yyyy-MM-dd'T'HH:mm:ssXXX, yyyy-MM-dd'T'HH:mm:ss]). Unknown relative time modifier string [31/31/2014:00:00:00].",
                         sqe.getMessage()
                 );
     }

src/test/java/com/teragrep/pth10/relativeTimeTest.java

@@ -336,7 +336,7 @@ public void parseTimestampLatestRelativeTestWithPlus() {
     )
     public void parseTimestampLatestRelativeTestWithoutSign() {
         String q = "index=cinnamon latest=3h ";
-        String expected = "Error parsing <3h>. Check that the timestamp or the relative time value is in the correct format (Supported timestamp formats <MM/dd/yyyy:HH:mm:ss>, <yyyy-MM-dd'T'HH:mm:ss>, <yyyy-MM-dd'T'HH:mm:ssXXX>). Unknown relative time modifier string [3h].";
+        String expected = "Error parsing <3h>. Check that the timestamp or the relative time value is in the correct format (Supported timestamp formats: [MM/dd/yyyy:HH:mm:ss, yyyy-MM-dd'T'HH:mm:ss.SSSXXX, yyyy-MM-dd'T'HH:mm:ss.SSS, yyyy-MM-dd'T'HH:mm:ssXXX, yyyy-MM-dd'T'HH:mm:ss]). Unknown relative time modifier string [3h].";
 
         RuntimeException exception = this.streamingTestUtil
                 .performThrowingDPLTest(RuntimeException.class, q, this.testFile, res -> {
@@ -437,7 +437,7 @@ public void parseTimestampEarliestRelativeSnapToDayLatestNow() {
     public void parseTimestampRelativeInvalidSnapToTimeUnitTest() {
         // pth10 ticket #66 query: 'index=... sourcetype=... earliest=@-5h latest=@-3h'
         String query = "index=cinnamon earliest=\"@-5h\" latest=\"@-3h\"";
-        String expected = "Error parsing <@-5h>. Check that the timestamp or the relative time value is in the correct format (Supported timestamp formats <MM/dd/yyyy:HH:mm:ss>, <yyyy-MM-dd'T'HH:mm:ss>, <yyyy-MM-dd'T'HH:mm:ssXXX>). Unknown relative time modifier string [@-5h].";
+        String expected = "Error parsing <@-5h>. Check that the timestamp or the relative time value is in the correct format (Supported timestamp formats: [MM/dd/yyyy:HH:mm:ss, yyyy-MM-dd'T'HH:mm:ss.SSSXXX, yyyy-MM-dd'T'HH:mm:ss.SSS, yyyy-MM-dd'T'HH:mm:ssXXX, yyyy-MM-dd'T'HH:mm:ss]). Unknown relative time modifier string [@-5h].";
 
         RuntimeException exception = this.streamingTestUtil
                 .performThrowingDPLTest(RuntimeException.class, query, this.testFile, res -> {