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

Highly optimize Varint21FrameDecoder #3792

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Outfluencer
Copy link
Collaborator

Removed byte[] alloc and reduce loops by using ByteBuf.forEachByte only ones and than parsing the Varint with bitshifting from the buffer

Removed byte[] alloc and reduce loops by using ByteBuf.forEachByte only ones and than parsing the Varint with bitshifting from the buffer
@Outfluencer
Copy link
Collaborator Author

can we also remove the direct IO buffers info?

@Outfluencer
Copy link
Collaborator Author

That is extremly disrespectfull

i have written the code myself it took me about a hour

@Outfluencer
Copy link
Collaborator Author

That is extremly disrespectfull
i have written the code myself it took me about a hour

Your code reminded me about one of the many velocity varint21 frame decoder variations it has seen over time. Sorry that I jumped to quickly to this conclusion, did not checked this and just wrote too early.

Edit: I do not agree with your opinion, that my comment was disrespectful.

Simply assuming that the code is copied and asking for the license from the rightful owner is disrespectful in my opinion.

@Outfluencer
Copy link
Collaborator Author

Also if it wasn‘t disrespectful why delete it

@Janmm14

This comment was marked as off-topic.

@Outfluencer
Copy link
Collaborator Author

Outfluencer commented Feb 20, 2025

i think i got a too good view on you with how u actually do useful pr's here, unfortunaly it seems you & your friend circle still miss some crucial skills. i will stop interacting with you starting now

I actually don't know what "friend circle" you mean and what "skills" i do miss, anyways this inappropriate on github.
I don't even think you know friends of mine.

It seems like you have a personal issue with me, or somebody i know. If so you got my discord tag, we can talk about it in private

@andreasdc
Copy link

Keep it friendly guys. By the way is it possible to add some security checks to the decoder with this patch?

@Outfluencer
Copy link
Collaborator Author

Keep it friendly guys. By the way is it possible to add some security checks to the decoder with this patch?

what do you mean by security checks

@xism4
Copy link
Contributor

xism4 commented Feb 20, 2025

i think i got a too good view on you with how u actually do useful pr's here, unfortunaly it seems you & your friend circle still miss some crucial skills. i will stop interacting with you starting now

I actually don't know what "friend circle" you mean and what "skills" i do miss, anyways this inappropriate on github. I don't even think you know friends of mine.

It seems like you have a personal issue with me, or somebody i know. If so you got my discord tag, we can talk about it in private

And if it's not the skills, it's your lack of veracity.

@andreasdc
Copy link

Keep it friendly guys. By the way is it possible to add some security checks to the decoder with this patch?

what do you mean by security checks

Some limiters for example, so it can't be bypassed by some naughty users.

@Outfluencer
Copy link
Collaborator Author

Outfluencer commented Feb 21, 2025

Keep it friendly guys. By the way is it possible to add some security checks to the decoder with this patch?

what do you mean by security checks

Some limiters for example, so it can't be bypassed by some naughty users.

I dont understand what can be bypassed here, and i dont know what you mean by limits, its a Varint21FrameDecoder, the limit is in the name of the decoder

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

Successfully merging this pull request may close these issues.

4 participants