powertools-lambda-python icon indicating copy to clipboard operation
powertools-lambda-python copied to clipboard

Maintenance: Deprecate `tracing.base.BaseProvider`

Open roger-zhangg opened this issue 1 year ago • 1 comments

Why is this needed?

In PR #2342 we are providing a observability provider friendly tracing provider to deprecate the current X-Ray specific provider tracing.base.baseprovider and tracing.base.BaseSegment. This new provider solution will enable our customer to send traces to Third party observability provider with ease. After #2342 is merged, the current provider will no longer be used in Tracer.

Which area does this relate to?

Tracer

Solution

This Deprecation is straightforward. This provider class is mostly for internal purpose and X-ray only. If there are customers developing their own provider based on this base provider, they will be able to migrate to the new provider proposed in #2342 with few method/class name changes

Acknowledgment

roger-zhangg avatar Apr 10 '24 16:04 roger-zhangg

Thank you for opening this @roger-zhangg ! Adding tags to track it for Powertools v3

rubenfonseca avatar Apr 11 '24 10:04 rubenfonseca

We won't deprecate those classes in V3. We will keep as is but instead we'll open doors to new providers.

We can plan to remove this in V4, but wee ned to plan this in advanced.

leandrodamascena avatar Aug 22 '24 16:08 leandrodamascena

⚠️COMMENT VISIBILITY WARNING⚠️

This issue is now closed. Please be mindful that future comments are hard for our team to see.

If you need more assistance, please either tag a team member or open a new issue that references this one.

If you wish to keep having a conversation with other community members under this issue feel free to do so.

github-actions[bot] avatar Aug 22 '24 16:08 github-actions[bot]