lxy/redkale
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

优化Response.channel.write

Browse Source
This commit is contained in:
Redkale
2023-01-01 17:51:52 +08:00
parent b3862cea72
commit 1c1211a298
2 changed files with 30 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/main/java/org/redkale/net/AsyncNioTcpProtocolServer.java
View File

@@ -107,8 +107,9 @@ class AsyncNioTcpProtocolServer extends ProtocolServer {
return pool == null ? safeResponsePool.get() : pool.get(); return pool == null ? safeResponsePool.get() : pool.get();
}; };
this.responseConsumer = (v) -> { this.responseConsumer = (v) -> {
if (Thread.currentThread() != v.thread && v.thread != null) { WorkThread thread = v.channel != null ? v.channel.getAsyncIOThread() : v.thread;
v.thread.execute(() -> { if (thread != null && !thread.inCurrThread()) {
thread.execute(() -> {
ObjectPool<Response> pool = localResponsePool.get(); ObjectPool<Response> pool = localResponsePool.get();
(pool == null ? safeResponsePool : pool).accept(v); (pool == null ? safeResponsePool : pool).accept(v);
}); });

31
src/main/java/org/redkale/net/Response.java
View File

@@ -47,6 +47,8 @@ public abstract class Response<C extends Context, R extends Request<C>> {
protected Servlet<C, R, ? extends Response<C, R>> servlet; protected Servlet<C, R, ? extends Response<C, R>> servlet;
protected final ByteBuffer writeBuffer;
private final CompletionHandler finishBytesHandler = new CompletionHandler<Integer, Void>() { private final CompletionHandler finishBytesHandler = new CompletionHandler<Integer, Void>() {
@Override @Override
@@ -65,13 +67,21 @@ public abstract class Response<C extends Context, R extends Request<C>> {
@Override @Override
public void completed(Integer result, ByteBuffer attachment) { public void completed(Integer result, ByteBuffer attachment) {
channel.offerBuffer(attachment); if (attachment != writeBuffer) {
channel.offerBuffer(attachment);
} else {
attachment.clear();
}
finish(); finish();
} }
@Override @Override
public void failed(Throwable exc, ByteBuffer attachment) { public void failed(Throwable exc, ByteBuffer attachment) {
channel.offerBuffer(attachment); if (attachment != writeBuffer) {
channel.offerBuffer(attachment);
} else {
attachment.clear();
}
finish(true); finish(true);
} }
@@ -105,6 +115,7 @@ public abstract class Response<C extends Context, R extends Request<C>> {
this.context = context; this.context = context;
this.request = request; this.request = request;
this.thread = WorkThread.currWorkThread(); this.thread = WorkThread.currWorkThread();
this.writeBuffer = context != null ? ByteBuffer.allocateDirect(context.getBufferCapacity()) : null;
} }
protected AsyncConnection removeChannel() { protected AsyncConnection removeChannel() {
@@ -217,7 +228,11 @@ public abstract class Response<C extends Context, R extends Request<C>> {
AsyncConnection conn = removeChannel(); AsyncConnection conn = removeChannel();
if (conn != null && conn.protocolCodec != null) { if (conn != null && conn.protocolCodec != null) {
this.responseConsumer.accept(this); this.responseConsumer.accept(this);
conn.read(conn.protocolCodec); if (conn.inCurrThread()) {
conn.read(conn.protocolCodec);
} else {
conn.execute(() -> conn.read(conn.protocolCodec));
}
} else { } else {
Supplier<Response> poolSupplier = this.responseSupplier; Supplier<Response> poolSupplier = this.responseSupplier;
Consumer<Response> poolConsumer = this.responseConsumer; Consumer<Response> poolConsumer = this.responseConsumer;
@@ -270,7 +285,15 @@ public abstract class Response<C extends Context, R extends Request<C>> {
} }
}); });
} else { } else {
this.channel.write(bs, offset, length, finishBytesHandler); ByteBuffer buffer = this.writeBuffer;
if (buffer != null && buffer.capacity() >= length) {
buffer.clear();
buffer.put(bs, offset, length);
buffer.flip();
this.channel.write(buffer, buffer, finishBufferHandler);
} else {
this.channel.write(bs, offset, length, finishBytesHandler);
}
} }
} }