kenhuuu commented on PR #2915: URL: https://github.com/apache/tinkerpop/pull/2915#issuecomment-2513108739
Based on information that I've recently read (which could be inaccurate), I think the `SO_KEEPALIVE` option and the `IdleTimeoutHandler` can be used together. TCP probes shouldn't be surfaced by `recv()` so it shouldn't reset the idle timer. If this can be confirmed (via a small test), then I think its probably OK to just add the `SO_KEEPALIVE` option regardless of whether the idle monitor is supported. That being said, I'm still a bit skeptical about the real-world usefulness of this PR given that Netty can't control things like keepalive period and so on as those require OS-level options to be set. This just ends up using OS defaults which users may or may not find useful. So while I still support merging this PR, I think it will ultimately be replaced by the idle monitor. Again, it would great if you could confirm that setting this option doesn't affect the idle monitor, in which case, you can just always have this option on. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
