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

buffer: simplify buffer API #3604

Open
jknack opened this issue Dec 23, 2024 · 1 comment
Open

buffer: simplify buffer API #3604

jknack opened this issue Dec 23, 2024 · 1 comment
Labels
break-change buffer
Milestone
4.0.0

Comments

@jknack
Copy link
Member

jknack commented Dec 23, 2024

  • simplify (probably remove) buffer API
    • We just need something that allow us to use netty buffers and/or undertow/jetty buffer pools
@jknack jknack added this to the 4.0.0 milestone Dec 23, 2024
@agentgt
Copy link
Contributor

agentgt commented Dec 23, 2024

@jknack let me know what you decide or link PRs if you can. I may have to adjust the jstachio module. I have a plethora of output options: https://jstach.io/doc/jstachio/current/apidocs/io.jstach.jstachio/io/jstach/jstachio/output/package-summary.html

Have a happy holidays!

jknack added a commit that referenced this issue Jan 5, 2025
@jknack
buffer: move actual send to the buffer implementation
d6222a0 
- remove the need of creating a buffer iterator for jetty and undertow
- ref #3604
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
break-change buffer
Projects
None yet
Milestone
4.0.0
Development

No branches or pull requests
2 participants
@agentgt @jknack