[tfdbg] Exclude DestroyResourceOp from debugger's dumping of eager op…

… execution

This removes a deadlock that happens in certain test configurations, e.g., DistributionStrategy / OneDeviceStrategy.

PiperOrigin-RevId: 284757549
Change-Id: I89f26718f4b3ec9c12ab867ef4ec1451c553f5d8

tensorflow/python/debug/lib/dumping_callback.py

@@ -467,10 +467,13 @@ def callback(self,
         return self._instrument_symbolic_tensors(
             outputs, op_type, op_name, context_id, output_tensor_ids)
     else:
-      if compat.as_bytes(op_type) == b"DebugNumericSummaryV2":
+      op_type_bytes = compat.as_bytes(op_type)
+      if op_type_bytes == b"DebugNumericSummaryV2":
         # TODO(b/140334369): Remove this special casing logic once op_callback.
         # automatically prevents infinite recursion in eager mode.
         return None
+      if op_type_bytes in op_callbacks_common.OP_CALLBACK_SKIP_OPS:
+        return None
       context_id = self._func_graph_id_from_func_name(op_type)
       input_ids = [t._id for t in inputs]  # pylint:disable=protected-access
       writer.WriteExecution(self._dump_eager_tensors(

tensorflow/python/debug/lib/op_callbacks_common.py

@@ -41,4 +41,6 @@
     b"TPUCompilationResult",
     b"TPUReplicatedOutput",
     b"ConfigureDistributedTPU",
+    # Other special ops used by TensorFlow internally.
+    b"DestroyResourceOp",
 )