Work around possible compiler bug causing bogus lifetime errors to be…

… printed occasionally (MystenLabs#2963)

Co-authored-by: Mark Logan <[email protected]>

crates/sui-storage/src/mutex_table.rs

@@ -59,7 +59,7 @@ impl<'b, K: Hash + 'b> MutexTable<K> {
         guards
     }
 
-    pub async fn acquire_lock<'a>(&'a self, k: &K) -> LockGuard<'a> {
+    pub async fn acquire_lock<'a, 'key>(&'a self, k: &'key K) -> LockGuard<'a> {
         let lock_idx = self.get_lock_idx(k);
         LockGuard(self.lock_table[lock_idx].lock().await)
     }

crates/sui-storage/src/write_ahead_log.rs

@@ -63,8 +63,11 @@ pub trait WriteAheadLog<'a, C> {
     ///
     ///   Err(e) => An error occurred.
     #[must_use]
-    async fn begin_tx(&'a self, tx: &TransactionDigest, cert: &C)
-        -> SuiResult<Option<Self::Guard>>;
+    async fn begin_tx<'b>(
+        &'a self,
+        tx: &'b TransactionDigest,
+        cert: &'b C,
+    ) -> SuiResult<Option<Self::Guard>>;
 
     /// Recoverable TXes are TXes that we find in the log at start up (which indicates we crashed
     /// while processing them) or implicitly dropped TXes (which can happen because we errored
@@ -258,10 +261,10 @@ where
 
     #[must_use]
     #[instrument(level = "debug", name = "begin_tx", skip_all)]
-    async fn begin_tx(
+    async fn begin_tx<'b>(
         &'a self,
-        tx: &TransactionDigest,
-        cert: &C,
+        tx: &'b TransactionDigest,
+        cert: &'b C,
     ) -> SuiResult<Option<DBTxGuard<'a, C>>> {
         let mutex_guard = self.mutex_table.acquire_lock(tx).await;
         trace!(digest = ?tx, "acquired tx lock");