diff --git a/gerrit-sshd/src/main/java/com/google/gerrit/sshd/BaseCommand.java b/gerrit-sshd/src/main/java/com/google/gerrit/sshd/BaseCommand.java index 6923ad18d3..634c47c8e6 100644 --- a/gerrit-sshd/src/main/java/com/google/gerrit/sshd/BaseCommand.java +++ b/gerrit-sshd/src/main/java/com/google/gerrit/sshd/BaseCommand.java @@ -369,6 +369,22 @@ public abstract class BaseCommand implements Command { } } + protected String getTaskDescription() { + StringBuilder m = new StringBuilder(); + m.append(context.getCommandLine()); + return m.toString(); + } + + private String getTaskName() { + StringBuilder m = new StringBuilder(); + m.append(getTaskDescription()); + if (user.isIdentifiedUser()) { + IdentifiedUser u = user.asIdentifiedUser(); + m.append(" (").append(u.getAccount().getUserName()).append(")"); + } + return m.toString(); + } + private final class TaskThunk implements CancelableRunnable, ProjectRunnable { private final CommandRunnable thunk; private final String taskName; @@ -376,14 +392,7 @@ public abstract class BaseCommand implements Command { private TaskThunk(CommandRunnable thunk) { this.thunk = thunk; - - StringBuilder m = new StringBuilder(); - m.append(context.getCommandLine()); - if (user.isIdentifiedUser()) { - IdentifiedUser u = user.asIdentifiedUser(); - m.append(" (").append(u.getAccount().getUserName()).append(")"); - } - this.taskName = m.toString(); + this.taskName = getTaskName(); } @Override diff --git a/gerrit-sshd/src/main/java/com/google/gerrit/sshd/commands/ReviewCommand.java b/gerrit-sshd/src/main/java/com/google/gerrit/sshd/commands/ReviewCommand.java index 2a82a26867..ff444e62ca 100644 --- a/gerrit-sshd/src/main/java/com/google/gerrit/sshd/commands/ReviewCommand.java +++ b/gerrit-sshd/src/main/java/com/google/gerrit/sshd/commands/ReviewCommand.java @@ -247,6 +247,11 @@ public class ReviewCommand extends SshCommand { } } + @Override + protected String getTaskDescription() { + return "gerrit review"; + } + private void applyReview(PatchSet patchSet, ReviewInput review) throws RestApiException { gApi.changes() .id(patchSet.getId().getParentKey().get())