[ 
https://issues.apache.org/jira/browse/FLINK-2624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15029868#comment-15029868
 ] 

ASF GitHub Bot commented on FLINK-2624:
---------------------------------------

Github user mxm commented on a diff in the pull request:

    https://github.com/apache/flink/pull/1243#discussion_r46046784
  
    --- Diff: 
flink-staging/flink-streaming/flink-streaming-connectors/flink-connector-rabbitmq/src/main/java/org/apache/flink/streaming/connectors/rabbitmq/RMQSource.java
 ---
    @@ -102,4 +116,18 @@ public void run(SourceContext<OUT> ctx) throws 
Exception {
        public void cancel() {
                running = false;
        }
    +
    +   @Override
    +   protected void acknowledgeIDs(List<Long> ids) {
    +           try {
    +                   channel.basicAck(ids.get(ids.size() - 1), true);
    --- End diff --
    
    Since we acknowledge all ids <= the last stored id (the true flag), it is 
sufficient to store only the last id. 


> RabbitMQ source / sink should participate in checkpointing
> ----------------------------------------------------------
>
>                 Key: FLINK-2624
>                 URL: https://issues.apache.org/jira/browse/FLINK-2624
>             Project: Flink
>          Issue Type: Bug
>          Components: Streaming Connectors
>    Affects Versions: 0.10.0
>            Reporter: Stephan Ewen
>            Assignee: Hilmi Yildirim
>
> The RabbitMQ connector does not offer any fault tolerance guarantees right 
> now, because it does not participate in the checkpointing.
> We should integrate it in a similar was as the {{FlinkKafkaConsumer}} is 
> integrated.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to