seatunnel
seatunnel copied to clipboard
[Bug][Connector-v2][RedisSink]Fix failures caused by different cluster mode operations.
Purpose of this pull request
If the redis mode is a cluster type, it will fail when operating the expire method. So different redis cluster types should declare and use corresponding objects.
Check list
- [ ] Code changed are covered with tests, or it does not need tests for reason:
- [ ] If any new Jar binary package adding in your PR, please add License Notice according New License Guide
- [ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
- [ ] If you are contributing the connector code, please check that the following files are updated:
- Update change log that in connector document. For more details you can refer to connector-v2
- Update plugin-mapping.properties and add new connector information in it
- Update the pom file of seatunnel-dist
- [ ] Update the
release-note
.
The code style should be fixed.
hello @lightzhao Can you provide v2 connector documentation for this expressing thanks
BTW, whether in cluster mode or standalone mode, all operations should be abstracted into upper level interfaces
Expire feature has some problems, I think this feature #4975 should revert and redesign it. For expire key, we should only operation once not every time after set operation.
What does it mean to operate only once, and how to implement only one operation?
BTW, whether in cluster mode or standalone mode, all operations should be abstracted into upper level interfaces
This looks good, abstract class + interface can implement different client operations.
hello @lightzhao Can you provide v2 connector documentation for this expressing thanks
ok, i try it.
I will redesign.