rusackas commented on PR #40425: URL: https://github.com/apache/superset/pull/40425#issuecomment-4736167459
@sha174n good catch on the newline payload — switched all the probes to `(exec 3<>/dev/tcp/...)` so it's a pure connect, no more `incomplete startup packet` noise. And yeah, the soft-ceiling caveat is real; for the connection-refused case it's fine, and I'd rather keep this dependency-free than reach for a `timeout` wrapper that isn't guaranteed present. Thanks for the review! -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
