ctubbsii commented on code in PR #3905:
URL: https://github.com/apache/accumulo/pull/3905#discussion_r1391559266
##########
shell/src/main/java/org/apache/accumulo/shell/commands/GrepCommand.java:
##########
@@ -49,6 +49,20 @@ public int execute(final String fullCommand, final
CommandLine cl, final Shell s
if (cl.getArgList().isEmpty()) {
throw new MissingArgumentException("No terms specified");
}
+ // Configure formatting options
+ final FormatterConfig config = new FormatterConfig();
+ config.setPrintTimestamps(cl.hasOption(timestampOpt.getOpt()));
Review Comment:
@rsingh433 I just noticed that this print timestamps feature is already
enabled on line 104, so this is redundant here.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
