java/com/google/gerrit/server/notedb/ChangeDraftNotesUpdate.java - gerrit.git - Git at Google

 // Copyright (C) 2014 The Android Open Source Project
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 // http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 // See the License for the specific language governing permissions and
 // limitations under the License.

 package com.google.gerrit.server.notedb;

 import static com.google.common.base.MoreObjects.firstNonNull;
 import static com.google.common.base.Preconditions.checkState;
 import static com.google.gerrit.server.logging.TraceContext.newTimer;
 import static org.eclipse.jgit.lib.Constants.OBJ_BLOB;

 import com.google.auto.value.AutoValue;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ListMultimap;
 import com.google.gerrit.common.Nullable;
 import com.google.gerrit.entities.Account;
 import com.google.gerrit.entities.Change;
 import com.google.gerrit.entities.Comment;
 import com.google.gerrit.entities.HumanComment;
 import com.google.gerrit.entities.Project;
 import com.google.gerrit.entities.RefNames;
 import com.google.gerrit.exceptions.StorageException;
 import com.google.gerrit.server.ChangeDraftUpdate;
 import com.google.gerrit.server.ChangeDraftUpdateExecutor;
 import com.google.gerrit.server.CurrentUser;
 import com.google.gerrit.server.GerritPersonIdent;
 import com.google.gerrit.server.config.AllUsersName;
 import com.google.gerrit.server.experiments.ExperimentFeatures;
 import com.google.gerrit.server.experiments.ExperimentFeaturesConstants;
 import com.google.gerrit.server.git.GitRepositoryManager;
 import com.google.gerrit.server.logging.Metadata;
 import com.google.gerrit.server.logging.TraceContext;
 import com.google.gerrit.server.query.change.ChangeNumberVirtualIdAlgorithm;
 import com.google.gerrit.server.update.BatchUpdateListener;
 import com.google.inject.Inject;
 import com.google.inject.assistedinject.Assisted;
 import com.google.inject.assistedinject.AssistedInject;
 import java.io.IOException;
 import java.time.Instant;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
 import org.eclipse.jgit.errors.ConfigInvalidException;
 import org.eclipse.jgit.lib.BatchRefUpdate;
 import org.eclipse.jgit.lib.CommitBuilder;
 import org.eclipse.jgit.lib.ObjectId;
 import org.eclipse.jgit.lib.ObjectInserter;
 import org.eclipse.jgit.lib.PersonIdent;
 import org.eclipse.jgit.lib.Ref;
 import org.eclipse.jgit.notes.NoteMap;
 import org.eclipse.jgit.revwalk.RevWalk;
 import org.eclipse.jgit.transport.PushCertificate;
 import org.eclipse.jgit.transport.ReceiveCommand;

 /**
  * A single delta to apply atomically to a change.
  *
  * <p>This delta contains only draft comments on a single patch set of a change by a single author.
  * This delta will become a single commit in the All-Users repository.
  *
  * <p>This class is not thread safe.
  */
 public class ChangeDraftNotesUpdate extends AbstractChangeUpdate implements ChangeDraftUpdate {
   private final ChangeNumberVirtualIdAlgorithm virtualIdFunc;

   public interface Factory extends ChangeDraftUpdateFactory {
     @Override
     ChangeDraftNotesUpdate create(
         ChangeNotes notes,
         @Assisted("effective") Account.Id accountId,
         @Assisted("real") Account.Id realAccountId,
         PersonIdent authorIdent,
         Instant when);

     @Override
     ChangeDraftNotesUpdate create(
         Change change,
         @Assisted("effective") Account.Id accountId,
         @Assisted("real") Account.Id realAccountId,
         PersonIdent authorIdent,
         Instant when);
   }

   @AutoValue
   abstract static class Key {
     abstract ObjectId commitId();

     abstract Comment.Key key();
   }

   enum DeleteReason {
     DELETED,
     PUBLISHED,
     FIXED
   }

   private static Key key(Comment c) {
     return new AutoValue_ChangeDraftNotesUpdate_Key(c.getCommitId(), c.key);
   }

   public static class Executor implements ChangeDraftUpdateExecutor, AutoCloseable {
     public interface Factory extends ChangeDraftUpdateExecutor.Factory<Executor> {
       @Override
       Executor create(CurrentUser currentUser);
     }

     private final GitRepositoryManager repoManager;
     private final AllUsersName allUsersName;
     private final NoteDbUpdateExecutor noteDbUpdateExecutor;
     private final CurrentUser currentUser;
     private final AllUsersAsyncUpdate updateAllUsersAsync;
     private OpenRepo allUsersRepo;
     private boolean shouldAllowFastForward = false;

     @Inject
     Executor(
         GitRepositoryManager repoManager,
         AllUsersName allUsersName,
         NoteDbUpdateExecutor noteDbUpdateExecutor,
         AllUsersAsyncUpdate updateAllUsersAsync,
         @Assisted CurrentUser currentUser) {
       this.updateAllUsersAsync = updateAllUsersAsync;
       this.repoManager = repoManager;
       this.allUsersName = allUsersName;
       this.noteDbUpdateExecutor = noteDbUpdateExecutor;
       this.currentUser = currentUser;
     }

     @Override
     public void queueAllDraftUpdates(ListMultimap<String, ChangeDraftUpdate> updaters)
         throws IOException {
       ListMultimap<String, ChangeDraftNotesUpdate> noteDbUpdaters =
           filterTypedUpdates(updaters, ChangeDraftNotesUpdate.class);
       if (canRunAsync(noteDbUpdaters.values())) {
         updateAllUsersAsync.setDraftUpdates(noteDbUpdaters);
       } else {
         initAllUsersRepoIfNull();
         shouldAllowFastForward = true;
         allUsersRepo.addUpdatesNoLimits(noteDbUpdaters);
       }
     }

     @Override
     public void queueDeletionForChangeDrafts(Change.Id id) throws IOException {
       initAllUsersRepoIfNull();
       // Just scan repo for ref names, but get "old" values from cmds.
       for (Ref r :
           allUsersRepo
               .repo
               .getRefDatabase()
               .getRefsByPrefix(RefNames.refsDraftCommentsPrefix(id))) {
         Optional<ObjectId> old = allUsersRepo.cmds.get(r.getName());
         old.ifPresent(
             objectId ->
                 allUsersRepo.cmds.add(
                     new ReceiveCommand(objectId, ObjectId.zeroId(), r.getName())));
       }
     }

     /**
      * Note this method does not fire {@link BatchUpdateListener#beforeUpdateRefs} events. However,
      * since the {@link BatchRefUpdate} object is returned, {@link
      * BatchUpdateListener#afterUpdateRefs} can be fired by the caller.
      */
     @Override
     public Optional<BatchRefUpdate> executeAllSyncUpdates(
         boolean dryRun, @Nullable PersonIdent refLogIdent, @Nullable String refLogMessage)
         throws IOException {
       if (allUsersRepo == null) {
         return Optional.empty();
       }
       try (TraceContext.TraceTimer ignored =
           newTimer("ChangeDraftNotesUpdate#Executor#updateAllUsersSync", Metadata.empty())) {
         return noteDbUpdateExecutor.execute(
             allUsersRepo,
             dryRun,
             shouldAllowFastForward,
             /* batchUpdateListeners= */ ImmutableList.of(),
             /* pushCert= */ null,
             refLogIdent,
             refLogMessage);
       }
     }

     @Override
     public void executeAllAsyncUpdates(
         @Nullable PersonIdent refLogIdent,
         @Nullable String refLogMessage,
         @Nullable PushCertificate pushCert) {
       updateAllUsersAsync.execute(refLogIdent, refLogMessage, pushCert, currentUser);
     }

     @Override
     public boolean isEmpty() {
       return (allUsersRepo == null || allUsersRepo.cmds.isEmpty()) && updateAllUsersAsync.isEmpty();
     }

     @Override
     public void close() throws Exception {
       if (allUsersRepo != null) {
         OpenRepo r = allUsersRepo;
         allUsersRepo = null;
         r.close();
       }
     }

     private void initAllUsersRepoIfNull() throws IOException {
       if (allUsersRepo == null) {
         allUsersRepo = OpenRepo.open(repoManager, allUsersName);
       }
     }
   }

   private final AllUsersName draftsProject;
   private final ExperimentFeatures experimentFeatures;

   private List<HumanComment> put = new ArrayList<>();
   private Map<Key, DeleteReason> delete = new HashMap<>();

   @SuppressWarnings("UnusedMethod")
   @AssistedInject
   private ChangeDraftNotesUpdate(
       @GerritPersonIdent PersonIdent serverIdent,
       AllUsersName allUsers,
       ChangeNoteUtil noteUtil,
       ExperimentFeatures experimentFeatures,
       @Nullable ChangeNumberVirtualIdAlgorithm virtualIdFunc,
       @Assisted ChangeNotes notes,
       @Assisted("effective") Account.Id accountId,
       @Assisted("real") Account.Id realAccountId,
       @Assisted PersonIdent authorIdent,
       @Assisted Instant when) {
     super(noteUtil, serverIdent, notes, null, accountId, realAccountId, authorIdent, when);
     this.draftsProject = allUsers;
     this.experimentFeatures = experimentFeatures;
     this.virtualIdFunc = virtualIdFunc;
   }

   @AssistedInject
   private ChangeDraftNotesUpdate(
       @GerritPersonIdent PersonIdent serverIdent,
       AllUsersName allUsers,
       ChangeNoteUtil noteUtil,
       ExperimentFeatures experimentFeatures,
       @Nullable ChangeNumberVirtualIdAlgorithm virtualIdFunc,
       @Assisted Change change,
       @Assisted("effective") Account.Id accountId,
       @Assisted("real") Account.Id realAccountId,
       @Assisted PersonIdent authorIdent,
       @Assisted Instant when) {
     super(noteUtil, serverIdent, null, change, accountId, realAccountId, authorIdent, when);
     this.draftsProject = allUsers;
     this.experimentFeatures = experimentFeatures;
     this.virtualIdFunc = virtualIdFunc;
   }

   @Override
   public void putDraftComment(HumanComment c) {
     checkState(!put.contains(c), "comment already added");
     verifyComment(c);
     put.add(c);
   }

   @Override
   public void markDraftCommentAsPublished(HumanComment c) {
     checkState(!delete.containsKey(key(c)), "comment already marked for deletion");
     verifyComment(c);
     delete.put(key(c), DeleteReason.PUBLISHED);
   }

   @Override
   public void addDraftCommentForDeletion(HumanComment c) {
     checkState(!delete.containsKey(key(c)), "comment already marked for deletion");
     verifyComment(c);
     delete.put(key(c), DeleteReason.DELETED);
   }

   @Override
   public void addAllDraftCommentsForDeletion(List<Comment> comments) {
     comments.forEach(
         comment -> {
           Key commentKey = key(comment);
           checkState(!delete.containsKey(commentKey), "comment already marked for deletion");
           delete.put(commentKey, DeleteReason.FIXED);
         });
   }

   /**
    * Returns whether all the updates in this instance can run asynchronously.
    *
    * <p>An update can run asynchronously only if it contains nothing but {@code PUBLISHED} or {@code
    * FIXED} draft deletions. User-initiated inversions/deletions must run synchronously in order to
    * return status.
    */
   @Override
   public boolean canRunAsync() {
     return put.isEmpty()
         && delete.values().stream()
             .allMatch(r -> r == DeleteReason.PUBLISHED || r == DeleteReason.FIXED);
   }

   /**
    * Returns a copy of the current {@link ChangeDraftNotesUpdate} that contains references to all
    * deletions. Copying of {@link ChangeDraftNotesUpdate} is only allowed if it contains no new
    * comments.
    */
   ChangeDraftNotesUpdate copy() {
     checkState(
         put.isEmpty(),
         "copying ChangeDraftNotesUpdate is allowed only if it doesn't contain new comments");
     ChangeDraftNotesUpdate clonedUpdate =
         new ChangeDraftNotesUpdate(
             authorIdent,
             draftsProject,
             noteUtil,
             experimentFeatures,
             virtualIdFunc,
             new Change(getChange()),
             accountId,
             realAccountId,
             authorIdent,
             when);
     clonedUpdate.delete.putAll(delete);
     return clonedUpdate;
   }

   @Nullable
   private CommitBuilder storeCommentsInNotes(
       RevWalk rw, ObjectInserter ins, ObjectId curr, CommitBuilder cb)
       throws ConfigInvalidException, IOException {
     RevisionNoteMap<ChangeRevisionNote> rnm = getRevisionNoteMap(rw, curr);
     RevisionNoteBuilder.Cache cache = new RevisionNoteBuilder.Cache(rnm);

     for (HumanComment c : put) {
       if (!experimentFeatures.isFeatureEnabled(
           ExperimentFeaturesConstants.ALLOW_FIX_SUGGESTIONS_IN_COMMENTS)) {
         checkState(c.fixSuggestions == null, "feature flag prohibits setting fixSuggestions");
       }
       if (!delete.keySet().contains(key(c))) {
         cache.get(c.getCommitId()).putComment(c);
       }
     }
     for (Key k : delete.keySet()) {
       cache.get(k.commitId()).deleteComment(k.key());
     }

     // keyed by commit ID.
     Map<ObjectId, RevisionNoteBuilder> builders = cache.getBuilders();
     boolean touchedAnyRevs = false;
     for (Map.Entry<ObjectId, RevisionNoteBuilder> e : builders.entrySet()) {
       ObjectId id = e.getKey();
       byte[] data = e.getValue().build(noteUtil.getChangeNoteJson());
       if (!Arrays.equals(data, e.getValue().baseRaw)) {
         touchedAnyRevs = true;
       }
       if (data.length == 0) {
         rnm.noteMap.remove(id);
       } else {
         ObjectId dataBlob = ins.insert(OBJ_BLOB, data);
         rnm.noteMap.set(id, dataBlob);
       }
     }

     // If we didn't touch any notes, tell the caller this was a no-op update. We
     // couldn't have done this in isEmpty() below because we hadn't read the old
     // data yet.
     if (!touchedAnyRevs) {
       return NO_OP_UPDATE;
     }

     // If there are no comments left, tell the
     // caller to delete the entire ref.
     if (!rnm.noteMap.iterator().hasNext()) {
       return null;
     }

     ObjectId treeId = rnm.noteMap.writeTree(ins);
     cb.setTreeId(treeId);
     return cb;
   }

   private RevisionNoteMap<ChangeRevisionNote> getRevisionNoteMap(RevWalk rw, ObjectId curr)
       throws ConfigInvalidException, IOException {
     // The old DraftCommentNotes already parsed the revision notes. We can reuse them as long as
     // the ref hasn't advanced.
     ChangeNotes changeNotes = getNotes();
     if (changeNotes != null) {
       DraftCommentNotes draftNotes = changeNotes.load().getDraftCommentNotes();
       if (draftNotes != null) {
         ObjectId idFromNotes = firstNonNull(draftNotes.getRevision(), ObjectId.zeroId());
         RevisionNoteMap<ChangeRevisionNote> rnm = draftNotes.getRevisionNoteMap();
         if (idFromNotes.equals(curr) && rnm != null) {
           return rnm;
         }
       }
     }
     NoteMap noteMap;
     if (!curr.equals(ObjectId.zeroId())) {
       noteMap = NoteMap.read(rw.getObjectReader(), rw.parseCommit(curr));
     } else {
       noteMap = NoteMap.newEmptyMap();
     }
     // Even though reading from changes might not be enabled, we need to
     // parse any existing revision notes, so we can merge them.
     return RevisionNoteMap.parse(
         noteUtil.getChangeNoteJson(), rw.getObjectReader(), noteMap, HumanComment.Status.DRAFT);
   }

   @Override
   protected CommitBuilder applyImpl(RevWalk rw, ObjectInserter ins, ObjectId curr)
       throws IOException {
     CommitBuilder cb = new CommitBuilder();
     cb.setMessage("Update draft comments");
     try {
       return storeCommentsInNotes(rw, ins, curr, cb);
     } catch (ConfigInvalidException e) {
       throw new StorageException(e);
     }
   }

   @Override
   protected Project.NameKey getProjectName() {
     return draftsProject;
   }

   @Override
   protected String getRefName() {
     return RefNames.refsDraftComments(getVirtualId(), accountId);
   }

   @Override
   public String getStorageKey() {
     return getRefName();
   }

   @Override
   protected void setParentCommit(CommitBuilder cb, ObjectId parentCommitId) {
     cb.setParentIds(); // Draft updates should not keep history of parent commits
   }

   @Override
   public boolean isEmpty() {
     return delete.isEmpty() && put.isEmpty();
   }

   private Change.Id getVirtualId() {
     Change change = getChange();
     return virtualIdFunc == null
         ? change.getId()
         : virtualIdFunc.apply(change.getServerId(), change.getId());
   }
 }
	// Copyright (C) 2014 The Android Open Source Project
	//
	// Licensed under the Apache License, Version 2.0 (the "License");
	// you may not use this file except in compliance with the License.
	// You may obtain a copy of the License at
	//
	// http://www.apache.org/licenses/LICENSE-2.0
	//
	// Unless required by applicable law or agreed to in writing, software
	// distributed under the License is distributed on an "AS IS" BASIS,
	// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
	// See the License for the specific language governing permissions and
	// limitations under the License.

	package com.google.gerrit.server.notedb;

	import static com.google.common.base.MoreObjects.firstNonNull;
	import static com.google.common.base.Preconditions.checkState;
	import static com.google.gerrit.server.logging.TraceContext.newTimer;
	import static org.eclipse.jgit.lib.Constants.OBJ_BLOB;

	import com.google.auto.value.AutoValue;
	import com.google.common.collect.ImmutableList;
	import com.google.common.collect.ListMultimap;
	import com.google.gerrit.common.Nullable;
	import com.google.gerrit.entities.Account;
	import com.google.gerrit.entities.Change;
	import com.google.gerrit.entities.Comment;
	import com.google.gerrit.entities.HumanComment;
	import com.google.gerrit.entities.Project;
	import com.google.gerrit.entities.RefNames;
	import com.google.gerrit.exceptions.StorageException;
	import com.google.gerrit.server.ChangeDraftUpdate;
	import com.google.gerrit.server.ChangeDraftUpdateExecutor;
	import com.google.gerrit.server.CurrentUser;
	import com.google.gerrit.server.GerritPersonIdent;
	import com.google.gerrit.server.config.AllUsersName;
	import com.google.gerrit.server.experiments.ExperimentFeatures;
	import com.google.gerrit.server.experiments.ExperimentFeaturesConstants;
	import com.google.gerrit.server.git.GitRepositoryManager;
	import com.google.gerrit.server.logging.Metadata;
	import com.google.gerrit.server.logging.TraceContext;
	import com.google.gerrit.server.query.change.ChangeNumberVirtualIdAlgorithm;
	import com.google.gerrit.server.update.BatchUpdateListener;
	import com.google.inject.Inject;
	import com.google.inject.assistedinject.Assisted;
	import com.google.inject.assistedinject.AssistedInject;
	import java.io.IOException;
	import java.time.Instant;
	import java.util.ArrayList;
	import java.util.Arrays;
	import java.util.HashMap;
	import java.util.List;
	import java.util.Map;
	import java.util.Optional;
	import org.eclipse.jgit.errors.ConfigInvalidException;
	import org.eclipse.jgit.lib.BatchRefUpdate;
	import org.eclipse.jgit.lib.CommitBuilder;
	import org.eclipse.jgit.lib.ObjectId;
	import org.eclipse.jgit.lib.ObjectInserter;
	import org.eclipse.jgit.lib.PersonIdent;
	import org.eclipse.jgit.lib.Ref;
	import org.eclipse.jgit.notes.NoteMap;
	import org.eclipse.jgit.revwalk.RevWalk;
	import org.eclipse.jgit.transport.PushCertificate;
	import org.eclipse.jgit.transport.ReceiveCommand;

	/**
	* A single delta to apply atomically to a change.
	*
	* <p>This delta contains only draft comments on a single patch set of a change by a single author.
	* This delta will become a single commit in the All-Users repository.
	*
	* <p>This class is not thread safe.
	*/
	public class ChangeDraftNotesUpdate extends AbstractChangeUpdate implements ChangeDraftUpdate {
	private final ChangeNumberVirtualIdAlgorithm virtualIdFunc;

	public interface Factory extends ChangeDraftUpdateFactory {
	@Override
	ChangeDraftNotesUpdate create(
	ChangeNotes notes,
	@Assisted("effective") Account.Id accountId,
	@Assisted("real") Account.Id realAccountId,
	PersonIdent authorIdent,
	Instant when);

	@Override
	ChangeDraftNotesUpdate create(
	Change change,
	@Assisted("effective") Account.Id accountId,
	@Assisted("real") Account.Id realAccountId,
	PersonIdent authorIdent,
	Instant when);
	}

	@AutoValue
	abstract static class Key {
	abstract ObjectId commitId();

	abstract Comment.Key key();
	}

	enum DeleteReason {
	DELETED,
	PUBLISHED,
	FIXED
	}

	private static Key key(Comment c) {
	return new AutoValue_ChangeDraftNotesUpdate_Key(c.getCommitId(), c.key);
	}

	public static class Executor implements ChangeDraftUpdateExecutor, AutoCloseable {
	public interface Factory extends ChangeDraftUpdateExecutor.Factory<Executor> {
	@Override
	Executor create(CurrentUser currentUser);
	}

	private final GitRepositoryManager repoManager;
	private final AllUsersName allUsersName;
	private final NoteDbUpdateExecutor noteDbUpdateExecutor;
	private final CurrentUser currentUser;
	private final AllUsersAsyncUpdate updateAllUsersAsync;
	private OpenRepo allUsersRepo;
	private boolean shouldAllowFastForward = false;

	@Inject
	Executor(
	GitRepositoryManager repoManager,
	AllUsersName allUsersName,
	NoteDbUpdateExecutor noteDbUpdateExecutor,
	AllUsersAsyncUpdate updateAllUsersAsync,
	@Assisted CurrentUser currentUser) {
	this.updateAllUsersAsync = updateAllUsersAsync;
	this.repoManager = repoManager;
	this.allUsersName = allUsersName;
	this.noteDbUpdateExecutor = noteDbUpdateExecutor;
	this.currentUser = currentUser;
	}

	@Override
	public void queueAllDraftUpdates(ListMultimap<String, ChangeDraftUpdate> updaters)
	throws IOException {
	ListMultimap<String, ChangeDraftNotesUpdate> noteDbUpdaters =
	filterTypedUpdates(updaters, ChangeDraftNotesUpdate.class);
	if (canRunAsync(noteDbUpdaters.values())) {
	updateAllUsersAsync.setDraftUpdates(noteDbUpdaters);
	} else {
	initAllUsersRepoIfNull();
	shouldAllowFastForward = true;
	allUsersRepo.addUpdatesNoLimits(noteDbUpdaters);
	}
	}

	@Override
	public void queueDeletionForChangeDrafts(Change.Id id) throws IOException {
	initAllUsersRepoIfNull();
	// Just scan repo for ref names, but get "old" values from cmds.
	for (Ref r :
	allUsersRepo
	.repo
	.getRefDatabase()
	.getRefsByPrefix(RefNames.refsDraftCommentsPrefix(id))) {
	Optional<ObjectId> old = allUsersRepo.cmds.get(r.getName());
	old.ifPresent(
	objectId ->
	allUsersRepo.cmds.add(
	new ReceiveCommand(objectId, ObjectId.zeroId(), r.getName())));
	}
	}

	/**
	* Note this method does not fire {@link BatchUpdateListener#beforeUpdateRefs} events. However,
	* since the {@link BatchRefUpdate} object is returned, {@link
	* BatchUpdateListener#afterUpdateRefs} can be fired by the caller.
	*/
	@Override
	public Optional<BatchRefUpdate> executeAllSyncUpdates(
	boolean dryRun, @Nullable PersonIdent refLogIdent, @Nullable String refLogMessage)
	throws IOException {
	if (allUsersRepo == null) {
	return Optional.empty();
	}
	try (TraceContext.TraceTimer ignored =
	newTimer("ChangeDraftNotesUpdate#Executor#updateAllUsersSync", Metadata.empty())) {
	return noteDbUpdateExecutor.execute(
	allUsersRepo,
	dryRun,
	shouldAllowFastForward,
	/* batchUpdateListeners= */ ImmutableList.of(),
	/* pushCert= */ null,
	refLogIdent,
	refLogMessage);
	}
	}

	@Override
	public void executeAllAsyncUpdates(
	@Nullable PersonIdent refLogIdent,
	@Nullable String refLogMessage,
	@Nullable PushCertificate pushCert) {
	updateAllUsersAsync.execute(refLogIdent, refLogMessage, pushCert, currentUser);
	}

	@Override
	public boolean isEmpty() {
	return (allUsersRepo == null \|\| allUsersRepo.cmds.isEmpty()) && updateAllUsersAsync.isEmpty();
	}

	@Override
	public void close() throws Exception {
	if (allUsersRepo != null) {
	OpenRepo r = allUsersRepo;
	allUsersRepo = null;
	r.close();
	}
	}

	private void initAllUsersRepoIfNull() throws IOException {
	if (allUsersRepo == null) {
	allUsersRepo = OpenRepo.open(repoManager, allUsersName);
	}
	}
	}

	private final AllUsersName draftsProject;
	private final ExperimentFeatures experimentFeatures;

	private List<HumanComment> put = new ArrayList<>();
	private Map<Key, DeleteReason> delete = new HashMap<>();

	@SuppressWarnings("UnusedMethod")
	@AssistedInject
	private ChangeDraftNotesUpdate(
	@GerritPersonIdent PersonIdent serverIdent,
	AllUsersName allUsers,
	ChangeNoteUtil noteUtil,
	ExperimentFeatures experimentFeatures,
	@Nullable ChangeNumberVirtualIdAlgorithm virtualIdFunc,
	@Assisted ChangeNotes notes,
	@Assisted("effective") Account.Id accountId,
	@Assisted("real") Account.Id realAccountId,
	@Assisted PersonIdent authorIdent,
	@Assisted Instant when) {
	super(noteUtil, serverIdent, notes, null, accountId, realAccountId, authorIdent, when);
	this.draftsProject = allUsers;
	this.experimentFeatures = experimentFeatures;
	this.virtualIdFunc = virtualIdFunc;
	}

	@AssistedInject
	private ChangeDraftNotesUpdate(
	@GerritPersonIdent PersonIdent serverIdent,
	AllUsersName allUsers,
	ChangeNoteUtil noteUtil,
	ExperimentFeatures experimentFeatures,
	@Nullable ChangeNumberVirtualIdAlgorithm virtualIdFunc,
	@Assisted Change change,
	@Assisted("effective") Account.Id accountId,
	@Assisted("real") Account.Id realAccountId,
	@Assisted PersonIdent authorIdent,
	@Assisted Instant when) {
	super(noteUtil, serverIdent, null, change, accountId, realAccountId, authorIdent, when);
	this.draftsProject = allUsers;
	this.experimentFeatures = experimentFeatures;
	this.virtualIdFunc = virtualIdFunc;
	}

	@Override
	public void putDraftComment(HumanComment c) {
	checkState(!put.contains(c), "comment already added");
	verifyComment(c);
	put.add(c);
	}

	@Override
	public void markDraftCommentAsPublished(HumanComment c) {
	checkState(!delete.containsKey(key(c)), "comment already marked for deletion");
	verifyComment(c);
	delete.put(key(c), DeleteReason.PUBLISHED);
	}

	@Override
	public void addDraftCommentForDeletion(HumanComment c) {
	checkState(!delete.containsKey(key(c)), "comment already marked for deletion");
	verifyComment(c);
	delete.put(key(c), DeleteReason.DELETED);
	}

	@Override
	public void addAllDraftCommentsForDeletion(List<Comment> comments) {
	comments.forEach(
	comment -> {
	Key commentKey = key(comment);
	checkState(!delete.containsKey(commentKey), "comment already marked for deletion");
	delete.put(commentKey, DeleteReason.FIXED);
	});
	}

	/**
	* Returns whether all the updates in this instance can run asynchronously.
	*
	* <p>An update can run asynchronously only if it contains nothing but {@code PUBLISHED} or {@code
	* FIXED} draft deletions. User-initiated inversions/deletions must run synchronously in order to
	* return status.
	*/
	@Override
	public boolean canRunAsync() {
	return put.isEmpty()
	&& delete.values().stream()
	.allMatch(r -> r == DeleteReason.PUBLISHED \|\| r == DeleteReason.FIXED);
	}

	/**
	* Returns a copy of the current {@link ChangeDraftNotesUpdate} that contains references to all
	* deletions. Copying of {@link ChangeDraftNotesUpdate} is only allowed if it contains no new
	* comments.
	*/
	ChangeDraftNotesUpdate copy() {
	checkState(
	put.isEmpty(),
	"copying ChangeDraftNotesUpdate is allowed only if it doesn't contain new comments");
	ChangeDraftNotesUpdate clonedUpdate =
	new ChangeDraftNotesUpdate(
	authorIdent,
	draftsProject,
	noteUtil,
	experimentFeatures,
	virtualIdFunc,
	new Change(getChange()),
	accountId,
	realAccountId,
	authorIdent,
	when);
	clonedUpdate.delete.putAll(delete);
	return clonedUpdate;
	}

	@Nullable
	private CommitBuilder storeCommentsInNotes(
	RevWalk rw, ObjectInserter ins, ObjectId curr, CommitBuilder cb)
	throws ConfigInvalidException, IOException {
	RevisionNoteMap<ChangeRevisionNote> rnm = getRevisionNoteMap(rw, curr);
	RevisionNoteBuilder.Cache cache = new RevisionNoteBuilder.Cache(rnm);

	for (HumanComment c : put) {
	if (!experimentFeatures.isFeatureEnabled(
	ExperimentFeaturesConstants.ALLOW_FIX_SUGGESTIONS_IN_COMMENTS)) {
	checkState(c.fixSuggestions == null, "feature flag prohibits setting fixSuggestions");
	}
	if (!delete.keySet().contains(key(c))) {
	cache.get(c.getCommitId()).putComment(c);
	}
	}
	for (Key k : delete.keySet()) {
	cache.get(k.commitId()).deleteComment(k.key());
	}

	// keyed by commit ID.
	Map<ObjectId, RevisionNoteBuilder> builders = cache.getBuilders();
	boolean touchedAnyRevs = false;
	for (Map.Entry<ObjectId, RevisionNoteBuilder> e : builders.entrySet()) {
	ObjectId id = e.getKey();
	byte[] data = e.getValue().build(noteUtil.getChangeNoteJson());
	if (!Arrays.equals(data, e.getValue().baseRaw)) {
	touchedAnyRevs = true;
	}
	if (data.length == 0) {
	rnm.noteMap.remove(id);
	} else {
	ObjectId dataBlob = ins.insert(OBJ_BLOB, data);
	rnm.noteMap.set(id, dataBlob);
	}
	}

	// If we didn't touch any notes, tell the caller this was a no-op update. We
	// couldn't have done this in isEmpty() below because we hadn't read the old
	// data yet.
	if (!touchedAnyRevs) {
	return NO_OP_UPDATE;
	}

	// If there are no comments left, tell the
	// caller to delete the entire ref.
	if (!rnm.noteMap.iterator().hasNext()) {
	return null;
	}

	ObjectId treeId = rnm.noteMap.writeTree(ins);
	cb.setTreeId(treeId);
	return cb;
	}

	private RevisionNoteMap<ChangeRevisionNote> getRevisionNoteMap(RevWalk rw, ObjectId curr)
	throws ConfigInvalidException, IOException {
	// The old DraftCommentNotes already parsed the revision notes. We can reuse them as long as
	// the ref hasn't advanced.
	ChangeNotes changeNotes = getNotes();
	if (changeNotes != null) {
	DraftCommentNotes draftNotes = changeNotes.load().getDraftCommentNotes();
	if (draftNotes != null) {
	ObjectId idFromNotes = firstNonNull(draftNotes.getRevision(), ObjectId.zeroId());
	RevisionNoteMap<ChangeRevisionNote> rnm = draftNotes.getRevisionNoteMap();
	if (idFromNotes.equals(curr) && rnm != null) {
	return rnm;
	}
	}
	}
	NoteMap noteMap;
	if (!curr.equals(ObjectId.zeroId())) {
	noteMap = NoteMap.read(rw.getObjectReader(), rw.parseCommit(curr));
	} else {
	noteMap = NoteMap.newEmptyMap();
	}
	// Even though reading from changes might not be enabled, we need to
	// parse any existing revision notes, so we can merge them.
	return RevisionNoteMap.parse(
	noteUtil.getChangeNoteJson(), rw.getObjectReader(), noteMap, HumanComment.Status.DRAFT);
	}

	@Override
	protected CommitBuilder applyImpl(RevWalk rw, ObjectInserter ins, ObjectId curr)
	throws IOException {
	CommitBuilder cb = new CommitBuilder();
	cb.setMessage("Update draft comments");
	try {
	return storeCommentsInNotes(rw, ins, curr, cb);
	} catch (ConfigInvalidException e) {
	throw new StorageException(e);
	}
	}

	@Override
	protected Project.NameKey getProjectName() {
	return draftsProject;
	}

	@Override
	protected String getRefName() {
	return RefNames.refsDraftComments(getVirtualId(), accountId);
	}

	@Override
	public String getStorageKey() {
	return getRefName();
	}

	@Override
	protected void setParentCommit(CommitBuilder cb, ObjectId parentCommitId) {
	cb.setParentIds(); // Draft updates should not keep history of parent commits
	}

	@Override
	public boolean isEmpty() {
	return delete.isEmpty() && put.isEmpty();
	}

	private Change.Id getVirtualId() {
	Change change = getChange();
	return virtualIdFunc == null
	? change.getId()
	: virtualIdFunc.apply(change.getServerId(), change.getId());
	}
	}