[ 
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)

Reply via email to