Use BoundedAttributes instead of raw dict to extract attributes from LogRecord

[findmyway]

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

Lines 336 to 339 in b184dc9

	def _get_attributes(record: logging.LogRecord) -> Attributes:
	attributes = {
	k: v for k, v in vars(record).items() if k not in _RESERVED_ATTRS
	}

In the latest main branch, attributes extracted from a LogRecord use dict directly. Better to use the BoundedAttributes from API to avoid some corner cases, like invalid value type or messages of long length.

[gdhuper]

/assign

[gdhuper]

@findmyway since I am new to this code base, I wanted to clarify something before I push out my PR.

In your description, you are suggesting to use BoundAttributes instead of a dict. Do you mean something like?

def _get_attributes(record: logging.LogRecord) -> BoundAttributes: 
       ...

[findmyway]

Yes, and I'd also suggest using BoundAttributes in the LogRecord definition.

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

Line 68 in b184dc9

attributes: Optional[Attributes] = None,

[srikanthccv]

@gdhuper are you still working on this?

[nstawski]

I would like to take this issue.

[gdhuper]

Sorry about the inactivity on this. I ran into some build errors that I couldn't get around. Feel free to take the issue.