commit 5c9df74813df5a7c878e524ba39a535a1d1ed5f0
author Sven Selberg <svense@axis.com> Wed Jan 09 13:30:36 2019 +0100
committer Sven Selberg <svense@axis.com> Wed Jan 09 13:30:36 2019 +0100
tree 6449e2e29da21d9ca93868145dab8dfe0aac8a67
parent 86bf28bc897b056d22bbdefe43a0ca4accd5783b

Remove getReviewDbProvider() from RequestContext

getReviewDbProvider() was removed from RequestContext in:
bd64916573 - Remove getReviewDbProvider() from RequestContext interface

Change-Id: If97fed6d174703a2c8373f3771acd7290f0872d9

src/main/java/com/googlesource/gerrit/plugins/rabbitmq/worker/UserEventWorker.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/rabbitmq/worker/UserEventWorker.java b/src/main/java/com/googlesource/gerrit/plugins/rabbitmq/worker/UserEventWorker.java
index de21369..034d5aa 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/rabbitmq/worker/UserEventWorker.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/rabbitmq/worker/UserEventWorker.java
@@ -18,7 +18,6 @@
 import com.google.gerrit.extensions.registration.DynamicSet;
 import com.google.gerrit.extensions.registration.RegistrationHandle;
 import com.google.gerrit.reviewdb.client.Account;
-import com.google.gerrit.reviewdb.server.ReviewDb;
 import com.google.gerrit.server.CurrentUser;
 import com.google.gerrit.server.IdentifiedUser;
 import com.google.gerrit.server.PluginUser;
@@ -29,10 +28,7 @@
 import com.google.gerrit.server.util.RequestContext;
 import com.google.gerrit.server.util.ThreadLocalRequestContext;
 import com.google.gwtorm.server.OrmException;
-import com.google.gwtorm.server.SchemaFactory;
 import com.google.inject.Inject;
-import com.google.inject.Provider;
-import com.google.inject.ProvisionException;
 import com.googlesource.gerrit.plugins.rabbitmq.message.Publisher;
 import java.io.IOException;
 import java.util.HashMap;
@@ -49,7 +45,6 @@
   private final IdentifiedUser.GenericFactory userFactory;
   private final ThreadLocalRequestContext threadLocalRequestContext;
   private final PluginUser pluginUser;
-  private final SchemaFactory<ReviewDb> schemaFactory;
   private final Map<Publisher, RegistrationHandle> eventListenerRegistrations;
 
   @Inject
@@ -59,15 +54,13 @@
       AccountResolver accountResolver,
       IdentifiedUser.GenericFactory userFactory,
       ThreadLocalRequestContext threadLocalRequestContext,
-      PluginUser pluginUser,
-      SchemaFactory<ReviewDb> schemaFactory) {
+      PluginUser pluginUser) {
     this.eventListeners = eventListeners;
     this.workQueue = workQueue;
     this.accountResolver = accountResolver;
     this.userFactory = userFactory;
     this.threadLocalRequestContext = threadLocalRequestContext;
     this.pluginUser = pluginUser;
-    this.schemaFactory = schemaFactory;
     eventListenerRegistrations = new HashMap<>();
   }
 
@@ -83,7 +76,6 @@
         .getDefaultQueue()
         .submit(
             new Runnable() {
-              private ReviewDb db;
               private Account userAccount;
 
               @Override
@@ -96,23 +88,6 @@
                           public CurrentUser getUser() {
                             return pluginUser;
                           }
-
-                          @Override
-                          public Provider<ReviewDb> getReviewDbProvider() {
-                            return new Provider<ReviewDb>() {
-                              @Override
-                              public ReviewDb get() {
-                                if (db == null) {
-                                  try {
-                                    db = schemaFactory.open();
-                                  } catch (OrmException e) {
-                                    throw new ProvisionException("Cannot open ReviewDb", e);
-                                  }
-                                }
-                                return db;
-                              }
-                            };
-                          }
                         });
                 try {
                   userAccount = accountResolver.find(userName);
@@ -142,10 +117,6 @@
                   return;
                 } finally {
                   threadLocalRequestContext.setContext(old);
-                  if (db != null) {
-                    db.close();
-                    db = null;
-                  }
                 }
               }
             });