Log when a Result is created from Error::ok (#8577)

Summary:

If this ctor is firing something has probably gone wrong. I found it a little hard to debug so adding logging to hopefully save someone else time in the future.

Differential Revision: D69870978

Author: JacobSzwejbka

runtime/core/result.h

@@ -60,7 +60,14 @@ class Result final {
    */
   /* implicit */ Result(Error error)
       : error_(error == Error::Ok ? Error::Internal : error),
-        hasValue_(false) {}
+        hasValue_(false) {
+          if ET_UNLIKELY(error == Error::Ok) {
+            ET_LOG(
+              Info, 
+              "Attempted to create Result from Error::OK, \
+              this has been converted to Error::Internal.");
+          }
+        }
 
   /// Value copy constructor.
   /* implicit */ Result(const T& val) : value_(val), hasValue_(true) {}