Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Update SDK time precision warning to reference correct PEP #1867

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Jun 1, 2021

Conversation

jlisee
Copy link
Contributor

@jlisee jlisee commented May 22, 2021

Description

When developing on Python older than 3.7 a warning is logged about timing
precision, this fixes that warning to reference PEP 564. This is the PEP
that talks about nanosecond precision in python:

https://www.python.org/dev/peps/pep-0564/

Type of change

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

How Has This Been Tested?

The above link has been read and verified it refers to time precision instead of
backporting SSL memory to Python 2.7 (PEP 546).

Checklist:

  • Documentation has been updated

Correct typo to reference the PIP that talks about nanosecond precision in python:
https://www.python.org/dev/peps/pep-0564/
@jlisee jlisee requested review from a team, aabmass and lzchen and removed request for a team May 22, 2021 20:17
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented May 22, 2021

CLA Signed

The committers are authorized under a signed CLA.

@owais
Copy link
Contributor

owais commented May 24, 2021

@jlisee Thanks for fixing this. Unfortunately we cannot merge until you sign the CLA

@lzchen lzchen added the Skip Changelog PRs that do not require a CHANGELOG.md entry label May 24, 2021
Copy link
Contributor

@ocelotl ocelotl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch!

@codeboten
Copy link
Contributor

Hey @jlisee, please take a look and sign the CLA to get this PR merged, thanks!

@jlisee
Copy link
Contributor Author

jlisee commented Jun 1, 2021

Hey @jlisee, please take a look and sign the CLA to get this PR merged, thanks!

I signed it as the email attached to this commit so we should be good to go. Thanks for the patience.

@lzchen lzchen merged commit 82b2e87 into open-telemetry:main Jun 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants