Skip to content
This repository was archived by the owner on Oct 19, 2023. It is now read-only.

Update flask_example.py#8

Closed
eddie-sumo wants to merge 1 commit intokinvolk:masterfrom
eddie-sumo:patch-1
Closed

Update flask_example.py#8
eddie-sumo wants to merge 1 commit intokinvolk:masterfrom
eddie-sumo:patch-1

Conversation

@eddie-sumo
Copy link

@eddie-sumo eddie-sumo commented Jan 20, 2023

Description

SimpleExportSpanProcessor has been renamed to SimpleSpanProcessor as of https://github.com/open-telemetry/opentelemetry-python/blob/main/CHANGELOG.md#version-100-2021-03-26
Minor update to new exporter name.

Fixes #
Traceback (most recent call last):
File "/Users/eddie/dev/trace_response/trace_response.py", line 9, in
from opentelemetry.sdk.trace.export import (
ImportError: cannot import name 'SimpleExportSpanProcessor' from 'opentelemetry.sdk.trace.export' (/opt/homebrew/lib/python3.10/site-packages/opentelemetry/sdk/trace/export/init.py)

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Tested on Python 3.10 running on MacOS Ventura with the latest OpenTelemetry SDK - this change was as of version 1.0 of the Opentelemetry SDK.

  • Test A

Checklist:

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added
  • Documentation has been updated

SimpleExportSpanProcessor has been renamed to SimpleSpanProcessor as of https://github.com/open-telemetry/opentelemetry-python/blob/main/CHANGELOG.md#version-100-2021-03-26
updated to new exporter name.
@eddie-sumo
Copy link
Author

Related PR for OpenTelemetry SDK change open-telemetry#1656

@mauriciovasquezbernal
Copy link
Member

Hey @eddie-sumo, this is a fork of https://github.com/open-telemetry/opentelemetry-python, please open this PR in the parent repository.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants