opendev/gerrit
Code Issues Proposed changes

Old DownloadCommand class must be used in intermediate upgrade phase

Browse Source 
To support zero downtime upgrade, move of user preferences data from
database to git backend is implemented in two phases. In second phase
the class AccountGeneralPreferences is dropped and replaced with
GeneralPreferencesInfo class. In DownloadConfig this replacement was
erroneously done already in the first phase: I218d9246. This broke
download-commands plugin. Restore the old class and postpose it
replacement in the phase 2 of migration.

Bug: Issue 3858
Change-Id: I70dbf4ce39239871fc2f67161f22eb402a83b2f5
This commit is contained in:
David Ostrovsky
2016-01-31 12:16:55 +01:00
committed by David Ostrovsky
parent 33543b7ed8
commit f8fb7f11eb
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-server/src/main/java/com/google/gerrit/server/config/DownloadConfig.java
View File

@@ -15,7 +15,7 @@
package com.google.gerrit.server.config;
import com.google.common.collect.ImmutableSet;
import com.google.gerrit.extensions.client.GeneralPreferencesInfo.DownloadCommand;
import com.google.gerrit.reviewdb.client.AccountGeneralPreferences.DownloadCommand;
import com.google.gerrit.reviewdb.client.CoreDownloadSchemes;
import com.google.gerrit.server.change.ArchiveFormat;
import com.google.inject.Inject;