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

Logger setup: WARNING: sun.reflect.Reflection.getCallerClass is not supported. This will impact performance. #3

Open
zerkowsm opened this issue May 9, 2022 · 2 comments

Comments

@zerkowsm
Copy link

zerkowsm commented May 9, 2022

Hello @maschnetwork

I'm still trying your setup to get some comparisons. I was trying to apply this to one of my apps. Unfortunately I got NoClassDefFoundError: com/amazonaws/services/lambda/runtime/LambdaLogger - apparently it doesn't like my logger setup. It was working properly without the instrumentation, though.
Anyway, I stepped back to the documentation and according to https://docs.aws.amazon.com/lambda/latest/dg/java-logging.html, this is the basic needed setup: https://github.com/awsdocs/aws-lambda-developer-guide/blob/main/sample-apps/java-basic/pom.xml (or here: https://github.com/awsdocs/aws-lambda-developer-guide/blob/main/sample-apps/blank-java/pom.xml) - does it still apply to custom runtimes?

Unfortunately it looks like I still miss some additional setup because with the instrumentation applied and the above setup I get: WARNING: sun.reflect.Reflection.getCallerClass is not supported. This will impact performance.

The app works fine but from the logs I can see it may impact the performance:

2022-05-09T11:29:15.751+02:00 START RequestId: 
2022-05-09T11:29:16.284+02:00 WARNING: sun.reflect.Reflection.getCallerClass is not supported. This will impact performance. | WARNING: sun.reflect.Reflection.getCallerClass is not supported. This will impact performance.
2022-05-09T11:29:19.403+02:00 END RequestId: 
2022-05-09T11:29:19.403+02:00 REPORT RequestId: Duration: 3650.94 ms	Billed Duration: 4473 ms Memory Size: 512 MB	Max Memory Used: 141 MB	Init Duration: 821.08 ms	XRAY TraceId: XXX Duration: 3650.94 ms Billed Duration: 4473 ms Memory Size: 512 MB Max Memory Used: 141 MB

Have you encountered this error/warning?

Regards
M

@maschnetwork
Copy link
Contributor

Hi @zerkowsm,

do you have an example repo for reference and to reproduce?

Best,
Max

@jkosternl
Copy link

See this issue for more information: aws/aws-lambda-java-libs#204

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants