[
https://issues.apache.org/jira/browse/BAHIR-95?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15923938#comment-15923938
]
ASF GitHub Bot commented on BAHIR-95:
-------------------------------------
Github user rmetzger commented on the issue:
https://github.com/apache/bahir-flink/pull/13
I think the reason why `ZREM` was not added is because this is the
`RedisSink`, which implies that you are writing data to redis, not removing it.
I'm not sure if it makes sense semantically. Do you have a use case for
removing data in the sink?
The change looks good overall.
Maybe you could also update the `README.md` file with the new command?
> Add ZREM to the Redis streaming connector in Flink
> --------------------------------------------------
>
> Key: BAHIR-95
> URL: https://issues.apache.org/jira/browse/BAHIR-95
> Project: Bahir
> Issue Type: New Feature
> Components: Flink Streaming Connectors
> Reporter: Aris Koliopoulos
> Priority: Minor
>
> It would be beneficial to add the ZREM command to the supported Redis commands
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
