InternalAccountQuery: Remove unused methods Change-Id: I45ccec5ed306e88a8ba1197402585e64986ab943 Signed-off-by: Edwin Kempin <ekempin@google.com>
diff --git a/java/com/google/gerrit/server/query/account/InternalAccountQuery.java b/java/com/google/gerrit/server/query/account/InternalAccountQuery.java index f1be580..02386ae 100644 --- a/java/com/google/gerrit/server/query/account/InternalAccountQuery.java +++ b/java/com/google/gerrit/server/query/account/InternalAccountQuery.java
@@ -17,11 +17,9 @@ import static java.util.stream.Collectors.toList; import static java.util.stream.Collectors.toSet; -import com.google.common.base.Joiner; import com.google.common.collect.ArrayListMultimap; import com.google.common.collect.ImmutableList; import com.google.common.collect.ImmutableListMultimap; -import com.google.common.collect.Lists; import com.google.common.collect.Multimap; import com.google.gerrit.index.FieldDef; import com.google.gerrit.index.IndexConfig; @@ -37,8 +35,6 @@ import java.util.Arrays; import java.util.List; import java.util.Set; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; /** * Query wrapper for the account index. @@ -47,8 +43,6 @@ * holding on to a single instance. */ public class InternalAccountQuery extends InternalQuery<AccountState> { - private static final Logger log = LoggerFactory.getLogger(InternalAccountQuery.class); - @Inject InternalAccountQuery( AccountQueryProcessor queryProcessor, @@ -94,28 +88,6 @@ return query(AccountPredicates.externalIdIncludingSecondaryEmails(externalId.toString())); } - public AccountState oneByExternalId(String externalId) throws OrmException { - return oneByExternalId(ExternalId.Key.parse(externalId)); - } - - public AccountState oneByExternalId(String scheme, String id) throws OrmException { - return oneByExternalId(ExternalId.Key.create(scheme, id)); - } - - public AccountState oneByExternalId(ExternalId.Key externalId) throws OrmException { - List<AccountState> accountStates = byExternalId(externalId); - if (accountStates.size() == 1) { - return accountStates.get(0); - } else if (accountStates.size() > 0) { - StringBuilder msg = new StringBuilder(); - msg.append("Ambiguous external ID ").append(externalId).append(" for accounts: "); - Joiner.on(", ") - .appendTo(msg, Lists.transform(accountStates, AccountState.ACCOUNT_ID_FUNCTION)); - log.warn(msg.toString()); - } - return null; - } - public List<AccountState> byFullName(String fullName) throws OrmException { return query(AccountPredicates.fullName(fullName)); }