[GitHub] [hadoop] xkrogen commented on a change in pull request #1749: HDFS-15044. [Dynamometer] Show the line of audit log when parsing it unsuccessfully.

[GitBox <gi...@apache.org>]

xkrogen commented on a change in pull request #1749: HDFS-15044. [Dynamometer] Show the line of audit log when parsing it unsuccessfully.
URL: https://github.com/apache/hadoop/pull/1749#discussion_r356123821
 
 

 ##########
 File path: hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-workload/src/main/java/org/apache/hadoop/tools/dynamometer/workloadgenerator/audit/AuditLogDirectParser.java
 ##########
 @@ -125,8 +125,9 @@ public AuditReplayCommand parse(Text inputLine,
       relativeTimestamp = dateFormat.parse(m.group("timestamp")).getTime()
           - startTimestamp;
     } catch (ParseException p) {
-      throw new IOException("Exception while parsing timestamp from audit log",
-          p);
+      throw new IOException(
+          "Exception while parsing timestamp from audit log line: `"
 
 Review comment:
   You have an extra backtick here. There should either be no backticks or one before and one after the `inputLine`.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org