From baece9823bc34c824fb72135d43584faab98b4c7 Mon Sep 17 00:00:00 2001 From: Greyson Parrelli Date: Tue, 14 Mar 2023 18:32:18 -0400 Subject: [PATCH] Remove log when enqueuing job within a transaction. Found the bug I put the logging in for, and now this log happens way to much after the decryption batching. --- .../org/thoughtcrime/securesms/jobmanager/JobManager.java | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/app/src/main/java/org/thoughtcrime/securesms/jobmanager/JobManager.java b/app/src/main/java/org/thoughtcrime/securesms/jobmanager/JobManager.java index 07507e5b8d..2fb20a5643 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/jobmanager/JobManager.java +++ b/app/src/main/java/org/thoughtcrime/securesms/jobmanager/JobManager.java @@ -61,12 +61,7 @@ public class JobManager implements ConstraintObserver.Notifier { public JobManager(@NonNull Application application, @NonNull Configuration configuration) { this.application = application; this.configuration = configuration; - this.executor = ThreadUtil.trace(new FilteredExecutor(configuration.getExecutorFactory().newSingleThreadExecutor("signal-JobManager"), () -> { - if (SignalDatabase.inTransaction()) { - Log.w(TAG, "Enqueuing a job while in a transaction!", new Throwable()); - } - return ThreadUtil.isMainThread(); - })); + this.executor = ThreadUtil.trace(new FilteredExecutor(configuration.getExecutorFactory().newSingleThreadExecutor("signal-JobManager"), ThreadUtil::isMainThread)); this.jobTracker = configuration.getJobTracker(); this.jobController = new JobController(application, configuration.getJobStorage(),