ASF GitHub Bot commented on BAHIR-95:

Github user rmetzger commented on the issue:

    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

Reply via email to