You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@camel.apache.org by "Claus Ibsen (JIRA)" <ji...@apache.org> on 2015/07/09 12:20:07 UTC

[jira] [Commented] (CAMEL-8639) Camel FTP component cannot recover after network failure

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

Claus Ibsen commented on CAMEL-8639:
------------------------------------

Can you work on a patch or something to better explain what you think needs to be fixed?

> Camel FTP component cannot recover after network failure
> --------------------------------------------------------
>
>                 Key: CAMEL-8639
>                 URL: https://issues.apache.org/jira/browse/CAMEL-8639
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-ftp
>            Reporter: Marcin MarzyƄski
>
> Camel FTP endpoint cannot recover by itself after network failure if no custom PollingConsumerPollStrategy is defined. Few years ago there was similar issue (https://issues.apache.org/jira/browse/CAMEL-1838), and it was fixed by adding in RemoteFileConsumer:
> this.setPollStrategy(new RemoteFilePollingConsumerPollStrategy());
> However it is broken again, as FtpEndpoint is overriding it with DefaultPollingConsumerPollStrategy in initConsumerProperties method, inherited from ScheduledPollEndpoint. I believe RemoteFilePollingConsumerPollStrategy should be used as default in RemoteFileEndpoint as well.
> Adding this.setPollStrategy(new RemoteFilePollingConsumerPollStrategy()); in RemoteFilePollingConsumerPollStrategy should fix this issue.
> Workaround for this bug is to create RemoteFilePollingConsumerPollStrategy in the registry and provide it directly to FTP endpoint by adding consumer.pollStrategy=#myRemoteFilePollingConsumerPollStrategyBeanId to the URL. This is also why FromFtpSimulateNetworkIssueRecoverTest didn't catch it - it's using custom PollingStrategy.



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