Replies: 1 comment
-
Can you expand and share your findings on the "significant allocation hotspot", like the profiling reports? |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Hi,
While profiling an application reading a non-trivial amount of data from S3, I noticed that the S3 checksum validation is a significant allocation hotspot. The following method in particular:
https://github.com/aws/aws-sdk-java-v2/blob/master/services/s3/src/main/java/software/amazon/awssdk/services/s3/internal/checksums/S3ChecksumValidatingPublisher.java#L81
On first read, it seems that these buffer copies could be avoided by working directly with ByteBuffer. Before I start experimenting with a patch along those lines, I'd like to know whether it's a “policy” requirement to use discrete buffers at each step in the publisher/subscriber chain, or whether a patch that avoids copies here could be accepted.
TIA
Beta Was this translation helpful? Give feedback.
All reactions