liunaijie commented on code in PR #8881:
URL: https://github.com/apache/seatunnel/pull/8881#discussion_r1983063072
##########
seatunnel-connectors-v2/connector-redis/src/main/java/org/apache/seatunnel/connectors/seatunnel/redis/config/RedisParameters.java:
##########
@@ -46,7 +46,7 @@
@Slf4j
public class RedisParameters implements Serializable {
private String host;
- private int port;
+ private Integer port;
Review Comment:
> > How about update it to redis default port 6379.
>
> In cluster mode it should be null.
In cluster moe, it won't be use
https://github.com/apache/seatunnel/pull/8841/files#diff-85c72df044b54265682571cdb07752eb396fdf68965cbcf863553efa9a22b34fR175
--
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: commits-unsubscr...@seatunnel.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
