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.

Sign up for GitHub

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

Jump to bottom

Allow this library to be used as a general parsing library #268

Merged

saartochner-lumigo merged 4 commits into master from RD-9866-use-as-library

Jan 22, 2023

Contributor

saartochner-lumigo commented Jan 22, 2023

Move the tracer-specific logic into a well-marked module and don't run code in global time if not lambda.

Created global_scope_exec to execute all the code in global time (only for lambda environments)
Refactor the reporter logic from the general utils to its specific module
All the global time code is now executed from the __init__ of the package instead of per-module


          move the tracer-specific logic into a well-marked module and don't ru…

ef46493

…n code in global time if not lambda

saartochner-lumigo requested a review from a team as a code owner

January 22, 2023 08:50

saartochner-lumigo commented

View reviewed changes

src/lumigo_tracer/lambda_tracer/lambda_reporter.py Show resolved Hide resolved

saartochner-lumigo commented

View reviewed changes

src/lumigo_tracer/lambda_tracer/lambda_reporter.py Show resolved Hide resolved

saartochner-lumigo commented

View reviewed changes

src/lumigo_tracer/lambda_tracer/tracer.py Show resolved Hide resolved

saartochner-lumigo commented

View reviewed changes

src/lumigo_tracer/lumigo_utils.py Show resolved Hide resolved

saartochner-lumigo commented

View reviewed changes

src/lumigo_tracer/lumigo_utils.py Show resolved Hide resolved

saartochner-lumigo commented

View reviewed changes

src/test/unit/lambda_tracer/test_lambda_reporter.py Show resolved Hide resolved

saartochner-lumigo commented

View reviewed changes

src/test/unit/test_lumigo_utils.py Show resolved Hide resolved

saartochner-lumigo commented

View reviewed changes

src/test/unit/wrappers/conftest.py Show resolved Hide resolved

saartochner-lumigo requested a review from GuyMoses

January 22, 2023 08:57

saartochner-lumigo added 2 commits

January 22, 2023 11:00


          move the tracer-specific logic into a well-marked module and don't ru…

0bdd12d

…n code in global time if not lambda


          move the tracer-specific logic into a well-marked module and don't ru…

bbf9b18

…n code in global time if not lambda

GuyMoses approved these changes

View reviewed changes


          support ReceiveMessage trigger in sqs parser

df455de

saartochner-lumigo merged commit c652338 into master

saartochner-lumigo deleted the RD-9866-use-as-library branch

January 22, 2023 13:00

saartochner-lumigo pushed a commit that referenced this pull request


          1.1.212 → 1.1.213. Changes: c652338 Allow this library to be used as …

a7b996f

…a general parsing library (#268)

1b791b6 layers-table: layers md [skip ci]

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

Labels

None yet