b717ced1cf
The large constant BATCH_SIZE was in part to avoid poor behavior on pushes of large numbers of changes, but those now use a bulk operation on RepoSequence anyway. For most usages we can default to a lower value, but still allow admins to raise it as necessary. Change-Id: I49edf1b9dd4b7d0aa47dd3f5eea489f8d88030f4
96 lines
2.9 KiB
Java
96 lines
2.9 KiB
Java
// Copyright (C) 2016 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;
|
|
|
|
import static com.google.common.base.Preconditions.checkArgument;
|
|
|
|
import com.google.common.annotations.VisibleForTesting;
|
|
import com.google.common.collect.ImmutableList;
|
|
import com.google.gerrit.reviewdb.server.ReviewDb;
|
|
import com.google.gerrit.server.config.AllProjectsName;
|
|
import com.google.gerrit.server.config.GerritServerConfig;
|
|
import com.google.gerrit.server.git.GitRepositoryManager;
|
|
import com.google.gerrit.server.notedb.NotesMigration;
|
|
import com.google.gerrit.server.notedb.RepoSequence;
|
|
import com.google.gwtorm.server.OrmException;
|
|
import com.google.inject.Inject;
|
|
import com.google.inject.Provider;
|
|
import com.google.inject.Singleton;
|
|
|
|
import org.eclipse.jgit.lib.Config;
|
|
|
|
import java.util.ArrayList;
|
|
import java.util.List;
|
|
|
|
@SuppressWarnings("deprecation")
|
|
@Singleton
|
|
public class Sequences {
|
|
private final Provider<ReviewDb> db;
|
|
private final NotesMigration migration;
|
|
private final RepoSequence changeSeq;
|
|
|
|
@Inject
|
|
Sequences(@GerritServerConfig Config cfg,
|
|
final Provider<ReviewDb> db,
|
|
NotesMigration migration,
|
|
GitRepositoryManager repoManager,
|
|
AllProjectsName allProjects) {
|
|
this.db = db;
|
|
this.migration = migration;
|
|
|
|
final int gap = cfg.getInt("noteDb", "changes", "initialSequenceGap", 0);
|
|
changeSeq = new RepoSequence(
|
|
repoManager,
|
|
allProjects,
|
|
"changes",
|
|
new RepoSequence.Seed() {
|
|
@Override
|
|
public int get() throws OrmException {
|
|
return db.get().nextChangeId() + gap;
|
|
}
|
|
},
|
|
cfg.getInt("noteDb", "changes", "sequenceBatchSize", 20));
|
|
}
|
|
|
|
public int nextChangeId() throws OrmException {
|
|
if (!migration.readChangeSequence()) {
|
|
return db.get().nextChangeId();
|
|
}
|
|
return changeSeq.next();
|
|
}
|
|
|
|
public ImmutableList<Integer> nextChangeIds(int count) throws OrmException {
|
|
if (migration.readChangeSequence()) {
|
|
return changeSeq.next(count);
|
|
}
|
|
|
|
if (count == 0) {
|
|
return ImmutableList.of();
|
|
}
|
|
checkArgument(count > 0, "count is negative: %s", count);
|
|
List<Integer> ids = new ArrayList<>(count);
|
|
ReviewDb db = this.db.get();
|
|
for (int i = 0; i < count; i++) {
|
|
ids.add(db.nextChangeId());
|
|
}
|
|
return ImmutableList.copyOf(ids);
|
|
}
|
|
|
|
@VisibleForTesting
|
|
public RepoSequence getChangeIdRepoSequence() {
|
|
return changeSeq;
|
|
}
|
|
}
|