Why lettuce not support connection heartbeat?

[jaybill-lee]

Feature Request

Redis Server has a "timeout" conf, when a connection idle "timeout" seconds, Redis Server will close the connection.

Why lettuce not support a connection heartbeat, such as:

ClientResources resources = ClientResources.builder()
                .nettyCustomizer(new NettyCustomizer() {
                    @Override
                    public void afterChannelInitialized(Channel channel) {
                        channel.pipeline().addLast(new IdleStateHandler(0, 3, 0, TimeUnit.SECONDS));
                        channel.pipeline().addLast(new ChannelDuplexHandler() {
                            @Override
                            public void userEventTriggered(ChannelHandlerContext ctx, Object evt) throws Exception {
                                if (evt instanceof IdleStateEvent) {
                                    Command<String, String, String> command = new Command<>(CommandType.PING, new StatusOutput<>(StringCodec.UTF8), null);
                                    if (ctx.channel().isActive()) {
                                        ctx.writeAndFlush(command);
                                    }
                                }
                            }
                        });
                    }
                }).build();

[mp911de]

There's no reason to have this as core feature. In fact, it just creates additional traffic for no good reason. Just because Redis has a feature to disconnect improperly implemented clients we don't need to investigate options for mitigation of the Redis feature.

[flyletc]

NettyCustomizer

Hello, I have also encountered this problem. May I ask if the issue has been resolved after adding a heartbeat