Memory allocation during LambdaEvent::new()

[timClicks]

We currently convert a &str to a String when a LambdaEvent is created. Presumably this doesn't need to be re-created each invocation of the Lambda function.

https://github.com/awslabs/aws-lambda-rust-runtime/blob/0f6e2a285677c00c35d65250166fa99cd56d3d68/lambda-extension/src/events.rs#L67

[greenwoodcm]

yeah doesn't seem to be necessary. do you think this ends up being a breaking change to the consumer?

[maxday]

👋 here is a proposal to fix this issue, coming with a breaking change unfortunately
I could not find a way to avoid copying and still using a String in LambdaEvent

[github-actions]

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for the maintainers of this repository to see.
If you need more assistance, please 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.

[timClicks]

Thanks for your input @maxday :D