Move the Optional out of LogAttributes

Author: xrmx

opentelemetry-api/src/opentelemetry/_logs/_internal/__init__.py

@@ -67,7 +67,7 @@ def __init__(
         severity_text: Optional[str] = None,
         severity_number: Optional[SeverityNumber] = None,
         body: AnyValue = None,
-        attributes: LogAttributes = None,
+        attributes: Optional[LogAttributes] = None,
     ):
         self.timestamp = timestamp
         if observed_timestamp is None:
@@ -90,7 +90,7 @@ def __init__(
         name: str,
         version: Optional[str] = None,
         schema_url: Optional[str] = None,
-        attributes: LogAttributes = None,
+        attributes: Optional[LogAttributes] = None,
     ) -> None:
         super().__init__()
         self._name = name
@@ -119,7 +119,7 @@ def __init__(  # pylint: disable=super-init-not-called
         name: str,
         version: Optional[str] = None,
         schema_url: Optional[str] = None,
-        attributes: LogAttributes = None,
+        attributes: Optional[LogAttributes] = None,
     ):
         self._name = name
         self._version = version
@@ -158,7 +158,7 @@ def get_logger(
         name: str,
         version: Optional[str] = None,
         schema_url: Optional[str] = None,
-        attributes: LogAttributes = None,
+        attributes: Optional[LogAttributes] = None,
     ) -> Logger:
         """Returns a `Logger` for use by the given instrumentation library.
 
@@ -196,7 +196,7 @@ def get_logger(
         name: str,
         version: Optional[str] = None,
         schema_url: Optional[str] = None,
-        attributes: LogAttributes = None,
+        attributes: Optional[LogAttributes] = None,
     ) -> Logger:
         """Returns a NoOpLogger."""
         return NoOpLogger(
@@ -210,7 +210,7 @@ def get_logger(
         name: str,
         version: Optional[str] = None,
         schema_url: Optional[str] = None,
-        attributes: LogAttributes = None,
+        attributes: Optional[LogAttributes] = None,
     ) -> Logger:
         if _LOGGER_PROVIDER:
             return _LOGGER_PROVIDER.get_logger(
@@ -273,7 +273,7 @@ def get_logger(
     instrumenting_library_version: str = "",
     logger_provider: Optional[LoggerProvider] = None,
     schema_url: Optional[str] = None,
-    attributes: LogAttributes = None,
+    attributes: Optional[LogAttributes] = None,
 ) -> "Logger":
     """Returns a `Logger` for use within a python process.
 

opentelemetry-api/src/opentelemetry/util/types.py

@@ -57,4 +57,4 @@
     ...,
 ]
 
-LogAttributes = Optional[Mapping[str, "AnyValue"]]
+LogAttributes = Mapping[str, "AnyValue"]

opentelemetry-api/tests/logs/test_proxy.py

@@ -28,7 +28,7 @@ def get_logger(
         name: str,
         version: typing.Optional[str] = None,
         schema_url: typing.Optional[str] = None,
-        attributes: LogAttributes = None,
+        attributes: typing.Optional[LogAttributes] = None,
     ) -> _logs.Logger:
         return LoggerTest(name)
 

opentelemetry-sdk/src/opentelemetry/sdk/_logs/_internal/__init__.py

@@ -181,7 +181,7 @@ def __init__(
         severity_number: Optional[SeverityNumber] = None,
         body: AnyValue = None,
         resource: Optional[Resource] = None,
-        attributes: LogAttributes = None,
+        attributes: Optional[LogAttributes] = None,
         limits: Optional[LogLimits] = _UnsetLogLimits,
     ):
         super().__init__(
@@ -633,7 +633,7 @@ def _get_logger_no_cache(
         name: str,
         version: Optional[str] = None,
         schema_url: Optional[str] = None,
-        attributes: LogAttributes = None,
+        attributes: Optional[LogAttributes] = None,
     ) -> Logger:
         return Logger(
             self._resource,
@@ -667,7 +667,7 @@ def get_logger(
         name: str,
         version: Optional[str] = None,
         schema_url: Optional[str] = None,
-        attributes: LogAttributes = None,
+        attributes: Optional[LogAttributes] = None,
     ) -> Logger:
         if self._disabled:
             warnings.warn("SDK is disabled.")