[oracledb/receiver] Removing the toLowercase on sql_text in topN & adding child_address into samples payload

[sv-splunk]

Description

Removing the toLowercase conversion on sql text in top_query payload. Also, adding child_address into the query_samples payload.

Link to tracking issue

Fixes #37478

Testing unit tests updated

Documentation updated

[crobert-1]

What's the reasoning behind removing the ToLower call?

Also, if it isn't too much work, it's best to have a PR for only one specific change at a time, rather than combining. This will allow each change to get merged as soon as it's ready, and the review process simpler. Not necessarily required, but if this PR doesn't get approved and merged quickly, you can try splitting it up for quicker progress.

[sv-splunk]

What's the reasoning behind removing the ToLower call?

ToLower is removed so the SQL query texts are further ingested in it's original form, which the users would prefer too. Also it helps with any further formatting in the downstream systems.

Also, if it isn't too much work, it's best to have a PR for only one specific change at a time, rather than combining. This will allow each change to get merged as soon as it's ready, and the review process simpler. Not necessarily required, but if this PR doesn't get approved and merged quickly, you can try splitting it up for quicker progress.

Agree!

@crobert-1