[PDS-510479] Table names must go through LoggingArgs (#7077)

`SnapshotTransaction`'s filtering limit now respects table logging safety correctly.

atlasdb-impl-shared/src/main/java/com/palantir/atlasdb/transaction/impl/SnapshotTransaction.java

@@ -1615,7 +1615,7 @@ private <T> ListenableFuture<Collection<Map.Entry<Cell, T>>> getWithPostFilterin
                                         + " or in the very rare case, could be due to transactions which constantly "
                                         + "conflict but never commit. These values will be cleaned up eventually, but"
                                         + " if the issue persists, ensure that sweep is caught up.",
-                                SafeArg.of("table", tableReference),
+                                LoggingArgs.tableRef(tableReference),
                                 SafeArg.of("maxIterations", MAX_POST_FILTERING_ITERATIONS));
                     }
                     snapshotEventRecorder.recordCellsReturned(tableReference, resultsAccumulator.size());

atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/transaction/impl/AbstractSnapshotTransactionTest.java

@@ -2998,7 +2998,7 @@ public void exceptionThrownWhenTooManyPostFilterIterationsOccur() {
                 .isInstanceOf(SafeIllegalStateException.class)
                 .hasMessageStartingWith("Unable to filter cells")
                 .hasExactlyArgs(
-                        SafeArg.of("table", TABLE_NO_SWEEP),
+                        UnsafeArg.of("tableRef", TABLE_NO_SWEEP.toString()),
                         SafeArg.of("maxIterations", SnapshotTransaction.MAX_POST_FILTERING_ITERATIONS));
     }
 

changelog/@unreleased/pr-7077.v2.yml

@@ -0,0 +1,6 @@
+type: fix
+fix:
+  description: '`SnapshotTransaction`''s filtering limit now respects table logging
+    safety correctly.'
+  links:
+  - https://github.com/palantir/atlasdb/pull/7077