Violeta Georgieva
Violeta Georgieva
@simonbasle We prepare separate PRs in very rare cases. For this one forward merge should be working.
@OlegDokuka There is a test failure. Can you take a look? Can you provide a test that shows the issue?
@OlegDokuka I rebased the changes and fixed the failing test. Once the CI is green we can merge this.
If I remember correctly this issue is related to logging usage https://stackoverflow.com/questions/63517483/spring-cloud-gateway-high-cpu-utilization-on-rhel-possible-to-switch-from-ep
@shakuzen This has been fixed with #2547
This is superseded by the new features in Reactor Core. If we need more we can reopen the issue.
I checked this and `channel.write` vs `channelContextHandler.write` is OK
> I'm not sure when I'll be able to get back to this. At this time, my team is not asking for it in the near-term. I'm fine closing this...
@yanwenliang Please provide reproducible example.
@yanwenliang Can you check whether you have the memory leak if you make this change in Spring Cloud Gateway https://github.com/spring-cloud/spring-cloud-gateway/blob/62e0f241d2aac61e699e4733ff2ed46c6865b045/spring-cloud-gateway-server/src/main/java/org/springframework/cloud/gateway/filter/NettyWriteResponseFilter.java#L124 `if (connection != null && connection.channel().isActive()) {`