We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Side effect : faild to failover. Root cause : Unnecessary reverse DNS lookup Bug fix : Avoid DNS lookup API.
@Override public void after(Object target, Object[] args, Object result, Throwable throwable) { ... final Operation op = (Operation) getOperation.invoke(target); if (op != null) { MemcachedNode handlingNode = op.getHandlingNode(); if (handlingNode != null) { SocketAddress socketAddress = handlingNode.getSocketAddress(); if (socketAddress instanceof InetSocketAddress) { InetSocketAddress address = (InetSocketAddress) socketAddress; // ROOT Cause : Unnecessary DNS look -> InetSocketAddress.getHostName() trace.recordEndPoint(address.getHostName() + ":" + address.getPort()); } } else { logger.info("no handling node"); } } else { logger.info("operation not found"); } }
The text was updated successfully, but these errors were encountered:
pinpoint-apm#1171 Fixed a side effect of Arcus interceptor
a79a27b
- modify dns lookup api InetSocketAddress.getHostName() -> InetAddress.getHostAddress()
Merge pull request #1173 from emeroad/#1171_fix_arcus_failover_sideeffct
42f7a20
#1171 Fixed a side effect of Arcus interceptor
pinpoint-apm#1174 Fixed a side effect of Arcus interceptor
3b160d1
- modify dns lookup api InetSocketAddress.getHostName() -> InetAddress.getHostAddress() pinpoint-apm#1171 porting
emeroad
No branches or pull requests
Side effect : faild to failover.
Root cause : Unnecessary reverse DNS lookup
Bug fix : Avoid DNS lookup API.
The text was updated successfully, but these errors were encountered: