From: Tim Otten <totten@civicrm.org>
Date: Thu, 2 Jun 2022 06:43:34 +0000 (-0700)
Subject: (NFC) Update stale comments
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=57fe3ccf15f943c4f889ce62063320b84529d889;p=civicrm-core.git

(NFC) Update stale comments
---

diff --git a/CRM/Upgrade/Incremental/php/FiveFiftyOne.php b/CRM/Upgrade/Incremental/php/FiveFiftyOne.php
index e0c8e44576..fe10b78e63 100644
--- a/CRM/Upgrade/Incremental/php/FiveFiftyOne.php
+++ b/CRM/Upgrade/Incremental/php/FiveFiftyOne.php
@@ -49,7 +49,7 @@ class CRM_Upgrade_Incremental_php_FiveFiftyOne extends CRM_Upgrade_Incremental_B
     // We don't have adequate data to differentiate these, so some will be wrong/suboptimal.
     // What's the impact of getting it wrong?
     // - For a finite/short-term queue, work has finished already (or will finish soon), so there is
-    //   very limited impact to wrongly setting `error=log`.
+    //   very limited impact to wrongly setting `error=delete`.
     // - For an infinite/long-term queue, work will continue indefinitely into the future. The impact
     //   of wrongly setting `error=abort` would continue indefinitely to the future.
     // Therefore, backfilling `error=log` is less-problematic than backfilling `error=abort`.
diff --git a/Civi.php b/Civi.php
index 7408a10f04..86e72ee847 100644
--- a/Civi.php
+++ b/Civi.php
@@ -117,7 +117,8 @@ class Civi {
    *   Specification for a queue.
    *   This is not required for accessing an existing queue.
    *   Specify this if you wish to auto-create the queue or to include advanced options (eg `reset`).
-   *   Example: ['type' => 'SqlParallel', 'error' => 'drop']
+   *   Example: ['type' => 'Sql', 'error' => 'abort']
+   *   Example: ['type' => 'SqlParallel', 'error' => 'delete']
    *   Defaults: ['reset'=>FALSE, 'is_persistent'=>TRUE, 'is_autorun'=>FALSE]
    * @return \CRM_Queue_Queue
    * @see \CRM_Queue_Service