[GitHub] [zookeeper] anmolnar commented on a change in pull request #1807: ZOOKEEPER-4461 Use logback in zookeeper-contrib project

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

anmolnar commented on a change in pull request #1807:
URL: https://github.com/apache/zookeeper/pull/1807#discussion_r794567897



##########
File path: zookeeper-contrib/zookeeper-contrib-loggraph/src/main/java/org/apache/zookeeper/graph/Log4JSource.java
##########
@@ -293,7 +293,7 @@ private void init() throws IOException {
 	if ((line != null) && (m = timep.matcher(line)).lookingAt()) {
 	    starttime = timestampFromText(dateformat, m.group(1));
 	} else {
-	    throw new IOException("Invalid log4j format. First line doesn't start with time");
+	    throw new IOException("Invalid log format. First line doesn't start with time");

Review comment:
       Yeah, I used the logback translator tool to convert the config file, so the format should be the same.
   Renaming the file also makes sense, I'll look into that. Thanks.




-- 
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: notifications-unsubscribe@zookeeper.apache.org

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