roryqi
roryqi
@maythorn300 Do you work on this PR?
Long time no actions. @mchades Maybe you can help this issue.
> [@jerqi](https://github.com/jerqi) Hi, I am willing to do this issue. Can you assign it to me! You can go ahead.
Actually I have a concern about the rest service configuration style. The last segment isn't camel style, is it ok?
You would better create an issue for a feature or a bug.
Maybe we shouldn't use PooledByteBuf. Because we will release the memory after we used the memory. You can see https://github.com/apache/incubator-uniffle/blob/28475fced12f5a4b79152d146ead1917982f44b9/storage/src/main/java/org/apache/uniffle/storage/handler/impl/DataSkippableReadHandler.java#L82 cc @maobaolong @advancedxy @rickyma @LuciferYang WDYT?
> Users can choose whatever kind of bytebuf they want: client: #1767 server: #2145 > > We basically will release every bytebuf, but it does not mean that we don't...
> I didn't look into the code carefully. I suppose this should be a bug. @maobaolong > > Similar to #1628? Yes, they are similar.
I suspect that the issue is caused by `TransportFrameDecoder` ``` @Override public void handlerRemoved(ChannelHandlerContext ctx) throws Exception { // Release all buffers that are still in our ownership. // Doing...