Fix code that ignores tensorflow::Status.

Change: 149170109

1 files changed, 9 insertions, 3 deletions

diff --git a/tensorflow/core/kernels/debug_ops.h b/tensorflow/core/kernels/debug_ops.h
index bfa163b292..bd42997732 100644
--- a/tensorflow/core/kernels/debug_ops.h
+++ b/tensorflow/core/kernels/debug_ops.h
@@ -94,9 +94,11 @@ class DebugIdentityOp : public OpKernel {
 
   void Compute(OpKernelContext* context) override {
     if (!debug_urls_.empty()) {
+      // TODO(b/32704451): Don't just ignore the ::tensorflow::Status object!
       DebugIO::PublishDebugTensor(tensor_name_, "DebugIdentity",
                                   context->input(0),
-                                  Env::Default()->NowMicros(), debug_urls_);
+                                  Env::Default()->NowMicros(), debug_urls_)
+          .IgnoreError();
     }
 
     context->set_output(0, context->input(0));
@@ -144,8 +146,10 @@ class DebugNanCountOp : public OpKernel {
     output_tensor->vec<int64>()(0) = nan_count;
 
     if (!debug_urls_.empty()) {
+      // TODO(b/32704451): Don't just ignore the ::tensorflow::Status object!
       DebugIO::PublishDebugTensor(tensor_name_, "DebugNanCount", *output_tensor,
-                                  Env::Default()->NowMicros(), debug_urls_);
+                                  Env::Default()->NowMicros(), debug_urls_)
+          .IgnoreError();
     }
   }
 
@@ -256,9 +260,11 @@ class DebugNumericSummaryOp : public OpKernel {
     output_tensor->vec<double>()(11) = variance;
 
     if (!debug_urls_.empty()) {
+      // TODO(b/32704451): Don't just ignore the ::tensorflow::Status object!
       DebugIO::PublishDebugTensor(tensor_name_, "DebugNumericSummary",
                                   *output_tensor, Env::Default()->NowMicros(),
-                                  debug_urls_);
+                                  debug_urls_)
+          .IgnoreError();
     }
   }