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.

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

Reduce data copying #30

Open
1 task done
wolfchimneyrock opened this issue Sep 11, 2023 · 0 comments
Open
1 task done

Reduce data copying #30

wolfchimneyrock opened this issue Sep 11, 2023 · 0 comments
Labels
A-Client Area: Java SDK bug Something isn't working

Comments

@wolfchimneyrock
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

Currently, there is a lot of time spent copying payload data.

Especially onerous is an entire copy made just to generate a crc32c of the payload.

Using the pack() / flush() API should be be more efficient.

Also, ByteBufferOutputStream copies incoming ByteBuffer contents to "pack" the data. Instead, the incoming ByteBuffers can be appended as-is.

bmq-sink-put

Expected Behavior

No response

Steps To Reproduce

put a high volume of messages into a queue while using Java Flight Recorder to profile the method execution times.

BlazingMQ Java SDK Version

0.0.9

Anything else?

No response
@wolfchimneyrock wolfchimneyrock added the bug Something isn't working label Sep 11, 2023
@wolfchimneyrock wolfchimneyrock mentioned this issue Sep 11, 2023
Open
@678098 678098 added the A-Client Area: Java SDK label Sep 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Client Area: Java SDK bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@678098 @wolfchimneyrock