Merge pull request #342 from benfred/develop

Don't write out debug messages on shutdown
mldrugdiscovery · Aug 6, 2018 · 63d363d · 63d363d
2 parents 727648b + 870ba57
commit 63d363d
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/python_bindings/nmslib.cc b/python_bindings/nmslib.cc
@@ -412,10 +412,12 @@ class PythonLogger
           inner.attr("critical")(message);
           break;
       }
-    } catch (const std::exception & e) {
-      std::cerr << "Failed to log '" << message << "'. Exception:" << e.what() << std::endl;
     } catch (...) {
-      std::cerr << "Failed to log '" << message << "'" << std::endl;
+      // This is almost certainly due to python process shut down.
+      // Just write the message out to stderr if its not a debug message
+      if (severity != LIB_DEBUG) {
+        StdErrLogger().log(severity, file, line, function, message);
+      }
     }
   }
 };