Precommit and comment changes

Author: DylanRussell

opentelemetry-sdk/src/opentelemetry/sdk/_shared_internal/__init__.py

@@ -40,7 +40,10 @@
 
 
 class DuplicateFilter(logging.Filter):
-    """This prevents logs generated when a log fails to be written to generate another log which fails to be written"""
+    """Filter that can be applied to internal `logger`'s.
+
+    Currently applied to `logger`s on the export logs path that could otherwise cause endless logging of errors or a
+    recursion depth exceeded issue in cases where logging itself results in an exception."""
 
     def filter(self, record):
         current_log = (

opentelemetry-sdk/tests/shared_internal/test_batch_processor.py

@@ -251,11 +251,12 @@ def test_shutdown_allows_1_export_to_finish(
         assert "Exception while exporting" in caplog.text
         assert 2 == exporter.num_export_calls
 
+
 class TestCommonFuncs(unittest.TestCase):
     def test_duplicate_logs_filter_works(self):
         test_logger = logging.getLogger("testLogger")
         test_logger.addFilter(DuplicateFilter())
         with self.assertLogs("testLogger") as cm:
             test_logger.info("message")
             test_logger.info("message")
-        self.assertEqual(len(cm.output), 1)
+        self.assertEqual(len(cm.output), 1)