opendev/gerrit
Code Issues Proposed changes

Merge "Fix plugin reloading for DynamicItems" into stable-2.10

Browse Source
This commit is contained in:
Saša Živkov
2015-01-22 10:30:31 +00:00
committed by Gerrit Code Review
parent 7406197bc9 d134e79fb1
commit caa4c0a8b5
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
gerrit-extension-api/src/main/java/com/google/gerrit/extensions/registration/DynamicItem.java
View File

@@ -184,7 +184,13 @@ public class DynamicItem<T> {
NamedProvider<T> old = null; NamedProvider<T> old = null;
while (!ref.compareAndSet(old, item)) { while (!ref.compareAndSet(old, item)) {
old = ref.get(); old = ref.get();
if (old != null && !"gerrit".equals(old.pluginName)) { if (old != null
&& !"gerrit".equals(old.pluginName)
&& !pluginName.equals(old.pluginName)) {
// We allow to replace:
// 1. Gerrit core items, e.g. websession cache
// can be replaced by plugin implementation
// 2. Reload of current plugin
throw new ProvisionException(String.format( throw new ProvisionException(String.format(
"%s already provided by %s, ignoring plugin %s", "%s already provided by %s, ignoring plugin %s",
this.key.getTypeLiteral(), old.pluginName, pluginName)); this.key.getTypeLiteral(), old.pluginName, pluginName));