seatunnel
seatunnel copied to clipboard
[Feature][Connector-v2][RedisSink]Support redis to set expiration time.
Purpose of this pull request
Support redis to set expiration time.
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
.
Does the parameter make sense?
Does the parameter make sense?
Surely it makes sense, if a key is created but no expiration is set for it, then the key will remain in memory forever. The production environment needs to set the expiration time, otherwise the memory will be greatly wasted.
@EricJoy2048 @TyrantLucifer @liugddx PTAL.
Can you add e2e test for this pr?
Can you add e2e test for this pr?
done.PTAL.
Wait for ci to complete
@TyrantLucifer
@liugddx @TaoZex PTAL.
@EricJoy2048 @TyrantLucifer @liugddx @TaoZex @Hisoka-X PTAL.
@TyrantLucifer
LGTM