Skip to content
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

Decouple processor needs to be added #842

Open
mcaparnacoxauto opened this issue Jan 29, 2024 · 7 comments
Open

Decouple processor needs to be added #842

mcaparnacoxauto opened this issue Jan 29, 2024 · 7 comments
Labels
enhancement New feature or request Lambda Lambda related issues

Comments

@mcaparnacoxauto
Copy link

Is your feature request related to a problem? Please describe.

We are currently exploring the possibility of utilizing the Decouple Processor, however, we've noticed that no processors are available as per the information provided on this link: aws-otel-lambda. This presents a challenge as the Decouple Processor offers unique features that could significantly enhance the performance of our Lambda layer.

Describe the solution you'd like

The Decouple Processor is specifically engineered to circumvent the limitations inherent in Lambda. It does this by reducing overhead and focusing on aspects that are directly relevant to Lambda, making it a more suitable choice compared to other processors. Therefore, it would be highly beneficial to include it in our setup.

One of the key advantages of the Decouple Processor is its awareness of the Lambda lifecycle. This means it can prevent the environment from being frozen or shut down until all pending traces/metrics/logs have been successfully exported. This feature alone could significantly boost the performance of the otel layer.

Furthermore, our Lambda is heavily utilized, processing millions of transactions. Without the Decouple Processor, these transactions could potentially slow down in the ADOT version, affecting overall performance.

Describe alternatives you've considered

We have considered switching to opentelemetry-lambda, but this alternative does not include the AWS services that we require.

In conclusion, the inclusion of the Decouple Processor in our setup could provide significant performance improvements, especially considering the high volume of transactions our Lambda processes. We believe this feature request is worth considering to enhance the efficiency and effectiveness of our operations.

@lukep-coxauto
Copy link

Hello, could we please get some comments by AWS on this issue? It is blocking us from using this layer because the processor is not in this layer, despite being added to the underlying opentelemetry-lambda framework that this layer is built on.

As an alternative, could someone provide clarification on the differences between using this ADOT OTEL layer and using opentelemetry-lambda directly?

@vasireddy99
Copy link
Contributor

vasireddy99 commented Apr 10, 2024

Hello, ADOT Lambda Layer consists a subset of components from the upstream and we sometimes apply patches(backport) as needed for fixes to avoid breaking changes etc., more information on ADOT Lambda here

@lukep-coxauto
Copy link

Hi @vasireddy99 thank you for the response!

Would you mind providing more direct answers to clarify the AWS team's stance on the following?

  1. Will the decouple processor ever be included in the aws-otel-lambda project, or is this out of scope for this project?

  2. Is there some type of comparison between using aws-otel-lambda and opentelemetry-lambda? I cannot tell if one is a superset of the other, or if they are divergent implementations. the README for this project says

As a downstream Repo of opentelemetry-lambda

which indicates to me that this repo is a derived repo based on the previous one.

  1. if we choose to use opentelemetry-lambda directly, are we losing any integration with Amazon functionality that the aws-otel-lambda layer provides? another way to phrase this would be, what is the reason to choose this project over the other one? the README states

aws-otel-lambda publishes AWS managed OpenTelemetry Lambda layers that are preconfigured for use with AWS services and bundle the reduced ADOT Collector. Users can onboard to OpenTelemetry in their existing Lambda functions by adding these ready-made layers directly.

but what does that mean for me? do I want the reduced collector? what preconfiguration with AWS services were done?

@vasireddy99
Copy link
Contributor

Hi Luke,

Will the decouple processor ever be included in the aws-otel-lambda project, or is this out of scope for this project?

We don't enough info about the timeline on when it will be included, cc: @mhausenblas

Is there some type of comparison between using aws-otel-lambda and opentelemetry-lambda? I cannot tell if one is a superset of the other, or if they are divergent implementations. the README for this project says

which indicates to me that this repo is a derived repo based on the previous one.

ADOT Lambda Layer consists a subset of components from the upstream and we sometimes apply patches(backport) as needed for fixes to avoid breaking changes etc., more information on ADOT Lambda here

yes we use opentelemetry-lambda repo as a git sub module

if we choose to use opentelemetry-lambda directly, are we losing any integration with Amazon functionality that the aws-otel-lambda layer provides?

For the java lambda layers, we made a patch to avoid this breaking change, other than that for the other layers, no integration functionality is lost in general when using otel layers.

if you are using the SDK layer with in-process exporter flushing the telemetry to the destination, reduced collector may not be necessary.

@mhausenblas
Copy link
Member

ADOT PM here. Thanks @vasireddy99 for providing some answers already and to add to this thread:

Will the decouple processor ever be included in the aws-otel-lambda project, or is this out of scope for this project?

At the current point in time it's not on our roadmap, but we're always looking for feedback (such as found in this convo) which then influences our prioritization.

if we choose to use opentelemetry-lambda directly, are we losing any integration with Amazon functionality that the aws-otel-lambda layer provides?

Technical or feature considerations aside: the same holds for the Lambda layer as is true for other integrations such as the EKS add-on (using the upstream OpenTelemetry Operator internally): when using ADOT and you're on AWS Enterprise Support , then you can get direct support from AWS, whereas if you're using upstream directly (such as layers here in this case or the OpenTelemetry Operator in the Kubernetes context) then the support you can expect is mainly: 1. community based, and 2. on a best-effort basis (by ourselves) here via GitHub.

@mhausenblas mhausenblas added enhancement New feature or request Lambda Lambda related issues labels Apr 11, 2024
@Dr-Emann
Copy link

Dr-Emann commented May 1, 2024

I believe related to #886 and #787

@cgilling
Copy link

cgilling commented May 2, 2024

I just want to add my voice to wanting to get this processor added, seems like it shouldn't be a lot of work to add it since its already a part of the collector. I'm stuck with not really being able to use this since I don't feel comfortable with the added latency that not having the decouple processor adds to user requests, but at the same time I can't use the awsxray trace support with the layers provided by the upstream repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Lambda Lambda related issues
Projects
None yet
Development

No branches or pull requests

6 participants