[pkg/translator/azure] Map application-defined properties

[joegoldman2]

Description:

#33583 introduced support for traces in Azure Event Hub receiver. However, application-defined properties have not been mapped. This PR fixes this and adds the mapping for properties.

Link to tracking Issue:

Testing:

Documentation:

cc @deepan10

[deepan10]

Hi @joegoldman2,
Thanks for the PR. I tested the implementation the application-defined properties are added into trace signal 👍

[github-actions]

This PR was marked stale due to lack of activity. It will be closed in 14 days.

[github-actions]

This PR was marked stale due to lack of activity. It will be closed in 14 days.

[github-actions]

This PR was marked stale due to lack of activity. It will be closed in 14 days.

[dmitryax]

@cparkins please review

[github-actions]

This PR was marked stale due to lack of activity. It will be closed in 14 days.

[github-actions]

This PR was marked stale due to lack of activity. It will be closed in 14 days.

[github-actions]

This PR was marked stale due to lack of activity. It will be closed in 14 days.

[andrzej-stencel]

Looks like this could use some unit tests. @joegoldman2 do you think you can add unit tests for the UnmarshalTraces method, similar to the UnmarshalLogs tests?

opentelemetry-collector-contrib/pkg/translator/azure/resourcelogs_to_logs_test.go

Line 384 in 499df64

func TestUnmarshalLogs(t *testing.T) {

[atoulme]

I agree a test would be nice. The change itself seems small enough it is ok to approve.