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

Claus Ibsen commented on CAMEL-3894:
------------------------------------

Tracy the file component is closely tied to the ftp component. So when you add 
an option/feature to the file component those options is often inherited by the 
ftp components. So its a good idea to:

- add unit test in camel-ftp as well for the new features
- document any API changes in the GenericFileConsumer as we have 3rd party end 
users who extend this component and do their custom file related components 
(add this to the API section in the release notes). You add a new parameter to 
the poll method. Also remember to add javadoc for that new option.
- Also consider if its possible to avoid tampering with the API so it can be 
backwards comp. out of the box. But sometimes the API must change and thats 
okay, if its documented in the release notes.
- you may need to adjust the code in camel-ftp for this new feature. I suggest 
you create a new JIRA ticket for that, so we get that included either in 2.8 or 
the next 2.9 release.
- You may want to add validation that max >= min. 

> Support maxdepth option on File Component when recursive option is enabled
> --------------------------------------------------------------------------
>
>                 Key: CAMEL-3894
>                 URL: https://issues.apache.org/jira/browse/CAMEL-3894
>             Project: Camel
>          Issue Type: New Feature
>          Components: camel-core
>    Affects Versions: 2.6.0
>            Reporter: Jesus CASTILLO
>             Fix For: Future
>
>
> It would be great if the File component adds support for the maxdepth option 
> just like the find command does, so we can prune the recursive directory 
> search when recursive is enabled.
> With maxdepth we can set the levels of Directories to descend during the 
> pollDirectory function.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to