rishabhdaim commented on code in PR #2038: URL: https://github.com/apache/jackrabbit-oak/pull/2038#discussion_r1959066346
########## oak-run/src/main/java/org/apache/jackrabbit/oak/run/RevisionsCommand.java: ########## @@ -191,10 +191,10 @@ private static class RevisionsOptions extends Utils.NodeStoreOptions { fullGcProgressSize = parser.accepts("fullGcProgressSize", "The number of documents to check for " + "garbage in each Full GC cycle") .withRequiredArg().ofType(Integer.class).defaultsTo(10000); - fullGcMaxAge = parser.accepts("fullGcMaxAge", "The maximum age of the document in seconds " + + fullGcMaxAgeSec = parser.accepts("fullGcMaxAge", "The maximum age of the document in seconds " + Review Comment: I would keep both variable name & param name same. ```suggestion fullGcMaxAge = parser.accepts("fullGcMaxAge", "The maximum age of the document in seconds " + ``` -- 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: oak-dev-unsubscr...@jackrabbit.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org