Remove confusing comment plus redundant binding in DefaultCommandModule

Remove confusing comment since command classes already tell such binding
code about being a master and/or slave command. Comment otherwise
becomes unnecessary as well as a maintenance trap -if any such master
/slave annotation changes in command class(es).

Remove redundant binding (duplicated line) for CreateAccountCommand
-still in DefaultCommandModule.

Change-Id: Ieaf8d98c951a3f471216cc651b22ad54eb6db6b6
This commit is contained in:
Marco Miller
2015-06-30 11:28:00 -04:00
parent 522e4f8bd4
commit a10d6b7244

View File

@@ -75,12 +75,12 @@ public class DefaultCommandModule extends CommandModule {
command("suexec").to(SuExec.class);
listener().to(ShowCaches.StartupListener.class);
// The following commands can only be run on a server in Master mode
command(gerrit, CreateAccountCommand.class);
command(gerrit, CreateGroupCommand.class);
command(gerrit, CreateProjectCommand.class);
command(gerrit, SetHeadCommand.class);
command(gerrit, AdminQueryShell.class);
if (slaveMode) {
command("git-receive-pack").to(NotSupportedInSlaveModeFailureCommand.class);
command("gerrit-receive-pack").to(NotSupportedInSlaveModeFailureCommand.class);
@@ -105,7 +105,6 @@ public class DefaultCommandModule extends CommandModule {
command(gerrit, SetAccountCommand.class);
command(gerrit, AdminSetParent.class);
command(gerrit, CreateAccountCommand.class);
command(testSubmit, TestSubmitRuleCommand.class);
command(testSubmit, TestSubmitTypeCommand.class);