Skip to content

Conversation

@therepanic
Copy link
Contributor

@therepanic therepanic commented Dec 1, 2025

Replaces EpollEventLoopGroup with MultiThreadIoEventLoopGroup configured via EpollIoHandler factory.

This aligns with the Netty 5 IO handler architecture and avoids usage of deprecated epoll transport APIs.

Replaces EpollEventLoopGroup with MultiThreadIoEventLoopGroup configured
via EpollIoHandler factory.

This aligns with the Netty 5 IO handler architecture and
avoids usage of deprecated epoll transport APIs.

Signed-off-by: Andrey Litvitski <[email protected]>
@therepanic
Copy link
Contributor Author

Btw (unrelated to the current change), for some reason I couldn't build the grpc-server-sample module (even without the current change), but when building with Actions, everything is fine. Here's part of the log: https://pastebin.com/7r1JQEMC

Copy link
Contributor

@onobc onobc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for another cleanup @therepanic !

@onobc
Copy link
Contributor

onobc commented Dec 2, 2025

Btw (unrelated to the current change), for some reason I couldn't build the grpc-server-sample module (even without the current change), but when building with Actions, everything is fine. Here's part of the log: https://pastebin.com/7r1JQEMC

I will investigate separately - thanks for the heads up!

Hi @therepanic , so this works for me locally and I am not sure where in the world it is getting ClientInterceptorsConfiguration but that will fail because that config does not exist and I am not sure it ever did? Do you see it anywhere if you global search your Spring gRPC repo? Try cleaning out your .m2 local repo for org/springframework/grpc and see if that fixes it?

@onobc onobc merged commit bd01b27 into spring-projects:main Dec 2, 2025
6 checks passed
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.

2 participants