[jira] Updated: (CAMEL-1241) Add camel-commons-vfs component
[ https://issues.apache.org/activemq/browse/CAMEL-1241?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Claus Ibsen updated CAMEL-1241: --- Fix Version/s: (was: 2.0.0) 2.1.0 > Add camel-commons-vfs component > --- > > Key: CAMEL-1241 > URL: https://issues.apache.org/activemq/browse/CAMEL-1241 > Project: Apache Camel > Issue Type: New Feature >Affects Versions: 2.0.0 >Reporter: Ramon Buckland >Assignee: Ramon Buckland > Fix For: 2.1.0 > > Attachments: CAMEL-1241.part_1_port_Generic_to_camel-core.diff, > CAMEL-1241_part2.zip, CAMEL-1241_part2_all_files.tgz, > commons-vfs-changes-part1.file-refactoring.diff > > Time Spent: 1 day, 2 hours > Remaining Estimate: 0 minutes > > Add a new component camel-commons-vfs which utilises the commons-vfs library > providing access to zip,jar,tgz,cifs,ftp and many other wonderful "file" type > resources. > The component will go into camel 2.0 as the new camel-file and camel-ftp > components have been re-written to support a better model. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (CAMEL-1241) Add camel-commons-vfs component
[ https://issues.apache.org/activemq/browse/CAMEL-1241?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ramon Buckland updated CAMEL-1241: -- Attachment: CAMEL-1241_part2_all_files.tgz Yes.. patch hell :-) Attached is all relevant new files or changes tar'd up. And below is the deletions needed svn rm components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/RemoteFileFilter.java svn rm components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/strategy/DeleteRemoteFileProcessStrategy.java svn rm components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/strategy/RemoteFileRenameExclusiveReadLockStrategy.java svn rm components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/strategy/RemoteFileExpressionRenamer.java svn rm components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/strategy/RemoteFileProcessStrategyFactory.java svn rm components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/strategy/RemoteFileRenamer.java svn rm components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/strategy/RenameRemoteFileProcessStrategy.java svn rm components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/strategy/NoOpRemoteFileProcessStrategy.java svn rm components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/strategy/DefaultRemoteFileRenamer.java svn rm components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/strategy/RemoteFileProcessStrategySupport.java svn rm components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/AntPathMatcherRemoteFileFilter.java > Add camel-commons-vfs component > --- > > Key: CAMEL-1241 > URL: https://issues.apache.org/activemq/browse/CAMEL-1241 > Project: Apache Camel > Issue Type: New Feature >Affects Versions: 2.0.0 >Reporter: Ramon Buckland >Assignee: Ramon Buckland > Fix For: 2.0.0 > > Attachments: CAMEL-1241.part_1_port_Generic_to_camel-core.diff, > CAMEL-1241_part2.zip, CAMEL-1241_part2_all_files.tgz, > commons-vfs-changes-part1.file-refactoring.diff > > Time Spent: 1 day, 2 hours > Remaining Estimate: 0 minutes > > Add a new component camel-commons-vfs which utilises the commons-vfs library > providing access to zip,jar,tgz,cifs,ftp and many other wonderful "file" type > resources. > The component will go into camel 2.0 as the new camel-file and camel-ftp > components have been re-written to support a better model. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (CAMEL-1241) Add camel-commons-vfs component
[ https://issues.apache.org/activemq/browse/CAMEL-1241?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ramon Buckland updated CAMEL-1241: -- Attachment: CAMEL-1241_part2.zip Fixed the getUriEndpointProtocol (removed from DefaultEndpoint) and changed to GenericFileEndpoint.getScheme() Broke patch into two camel-core camel-ftp (absolute or relative directory patch supplied) Both need to be applied > Add camel-commons-vfs component > --- > > Key: CAMEL-1241 > URL: https://issues.apache.org/activemq/browse/CAMEL-1241 > Project: Apache Camel > Issue Type: New Feature >Affects Versions: 2.0.0 >Reporter: Ramon Buckland >Assignee: Ramon Buckland > Fix For: 2.0.0 > > Attachments: CAMEL-1241.part_1_port_Generic_to_camel-core.diff, > CAMEL-1241_part2.zip, commons-vfs-changes-part1.file-refactoring.diff > > Time Spent: 1 day, 2 hours > Remaining Estimate: 0 minutes > > Add a new component camel-commons-vfs which utilises the commons-vfs library > providing access to zip,jar,tgz,cifs,ftp and many other wonderful "file" type > resources. > The component will go into camel 2.0 as the new camel-file and camel-ftp > components have been re-written to support a better model. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (CAMEL-1241) Add camel-commons-vfs component
[ https://issues.apache.org/activemq/browse/CAMEL-1241?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ramon Buckland updated CAMEL-1241: -- Attachment: (was: CAMEL-1241_Refactor_S-FTP_to_use_GenericYYY-v2.patch) > Add camel-commons-vfs component > --- > > Key: CAMEL-1241 > URL: https://issues.apache.org/activemq/browse/CAMEL-1241 > Project: Apache Camel > Issue Type: New Feature >Affects Versions: 2.0.0 >Reporter: Ramon Buckland >Assignee: Ramon Buckland > Fix For: 2.0.0 > > Attachments: CAMEL-1241.part_1_port_Generic_to_camel-core.diff, > commons-vfs-changes-part1.file-refactoring.diff > > Time Spent: 1 day, 2 hours > Remaining Estimate: 0 minutes > > Add a new component camel-commons-vfs which utilises the commons-vfs library > providing access to zip,jar,tgz,cifs,ftp and many other wonderful "file" type > resources. > The component will go into camel 2.0 as the new camel-file and camel-ftp > components have been re-written to support a better model. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (CAMEL-1241) Add camel-commons-vfs component
[ https://issues.apache.org/activemq/browse/CAMEL-1241?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ramon Buckland updated CAMEL-1241: -- Attachment: CAMEL-1241_Refactor_S-FTP_to_use_GenericYYY-v2.patch Fixed the patch (I had local uber pom.xml changes for future work not yet required) .. removed these. > Add camel-commons-vfs component > --- > > Key: CAMEL-1241 > URL: https://issues.apache.org/activemq/browse/CAMEL-1241 > Project: Apache Camel > Issue Type: New Feature >Affects Versions: 2.0.0 >Reporter: Ramon Buckland >Assignee: Ramon Buckland > Fix For: 2.0.0 > > Attachments: CAMEL-1241.part_1_port_Generic_to_camel-core.diff, > CAMEL-1241_Refactor_S-FTP_to_use_GenericYYY-v2.patch, > commons-vfs-changes-part1.file-refactoring.diff > > Time Spent: 1 day, 2 hours > Remaining Estimate: 0 minutes > > Add a new component camel-commons-vfs which utilises the commons-vfs library > providing access to zip,jar,tgz,cifs,ftp and many other wonderful "file" type > resources. > The component will go into camel 2.0 as the new camel-file and camel-ftp > components have been re-written to support a better model. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (CAMEL-1241) Add camel-commons-vfs component
[ https://issues.apache.org/activemq/browse/CAMEL-1241?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ramon Buckland updated CAMEL-1241: -- Attachment: (was: CAMEL-1241_Refactor_S-FTP_to_use_GenericYYY.patch) > Add camel-commons-vfs component > --- > > Key: CAMEL-1241 > URL: https://issues.apache.org/activemq/browse/CAMEL-1241 > Project: Apache Camel > Issue Type: New Feature >Affects Versions: 2.0.0 >Reporter: Ramon Buckland >Assignee: Ramon Buckland > Fix For: 2.0.0 > > Attachments: CAMEL-1241.part_1_port_Generic_to_camel-core.diff, > commons-vfs-changes-part1.file-refactoring.diff > > Time Spent: 1 day, 2 hours > Remaining Estimate: 0 minutes > > Add a new component camel-commons-vfs which utilises the commons-vfs library > providing access to zip,jar,tgz,cifs,ftp and many other wonderful "file" type > resources. > The component will go into camel 2.0 as the new camel-file and camel-ftp > components have been re-written to support a better model. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
Re: [jira] Updated: (CAMEL-1241) Add camel-commons-vfs component
> > Attachment: CAMEL-1241_Refactor_S-FTP_to_use_GenericYYY.patch > This is patch part 2 for the commons-vfs. My next step is (a) implement commons-vfs (camel-commons-vfs) (b) Fix CAMEL-1290 (s)ftp problem. I suspect it is not a biggy. before I do (a) and (b) .. I need a rest. nighto all.
[jira] Updated: (CAMEL-1241) Add camel-commons-vfs component
[ https://issues.apache.org/activemq/browse/CAMEL-1241?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ramon Buckland updated CAMEL-1241: -- Attachment: CAMEL-1241_Refactor_S-FTP_to_use_GenericYYY.patch This patch implements the GenericFile* classes / ifaces across the original Remote* classes. Specifically, the following occurs with this patch. (a) minor modifications to the new Generic* classes in camel-core (to support RemoteFile style endpoints) (b) added a method to DefaultEndpoint to obtain the "protocol" in the uri (so we can re-lookup our META-INF file to find default strategy) (c) refactored all the FTP tests to use target/ftpserver/home as the ftp server root, instead of res/ This patch readies the camel-core / camel-ftp (file based) so we can now implement commons-vfs into the fray. All Tests are working as of diff revision. > Add camel-commons-vfs component > --- > > Key: CAMEL-1241 > URL: https://issues.apache.org/activemq/browse/CAMEL-1241 > Project: Apache Camel > Issue Type: New Feature >Affects Versions: 2.0.0 >Reporter: Ramon Buckland >Assignee: Ramon Buckland > Fix For: 2.0.0 > > Attachments: CAMEL-1241.part_1_port_Generic_to_camel-core.diff, > CAMEL-1241_Refactor_S-FTP_to_use_GenericYYY.patch, > commons-vfs-changes-part1.file-refactoring.diff > > > Add a new component camel-commons-vfs which utilises the commons-vfs library > providing access to zip,jar,tgz,cifs,ftp and many other wonderful "file" type > resources. > The component will go into camel 2.0 as the new camel-file and camel-ftp > components have been re-written to support a better model. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (CAMEL-1241) Add camel-commons-vfs component
[
https://issues.apache.org/activemq/browse/CAMEL-1241?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Ramon Buckland updated CAMEL-1241:
--
Attachment: CAMEL-1241.part_1_port_Generic_to_camel-core.diff
As per Claus' suggestion. This is part 1 of the list below ..
So if you can provide diffs in multiple steps. A rough plan like this:
{quote}
1) Generic* that are used by the FTP component
2) When #1 is commited to triunk and you code is aligned with trunk,
then move on to
3) Move Generic* from camel-ftp to camel-core
4) When #3 is commited to triunk and you code is aligned with trunk,
then move on to
5) You can start on commons-vfs component
6) Then we can take a look at the java generics stuff and get that
done in the right way
7) Refactor File in camel-core to use the new generics
8) Add the new commons-vfs component when its ready and done
9) Add wiki doc for the new commons-vfs component
{quote}
> Add camel-commons-vfs component
> ---
>
> Key: CAMEL-1241
> URL: https://issues.apache.org/activemq/browse/CAMEL-1241
> Project: Apache Camel
> Issue Type: New Feature
>Affects Versions: 2.0.0
>Reporter: Ramon Buckland
>Assignee: Ramon Buckland
> Fix For: 2.0.0
>
> Attachments: CAMEL-1241.part_1_port_Generic_to_camel-core.diff,
> commons-vfs-changes-part1.file-refactoring.diff
>
>
> Add a new component camel-commons-vfs which utilises the commons-vfs library
> providing access to zip,jar,tgz,cifs,ftp and many other wonderful "file" type
> resources.
> The component will go into camel 2.0 as the new camel-file and camel-ftp
> components have been re-written to support a better model.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
Re: [jira] Updated: (CAMEL-1241) Add camel-commons-vfs component
On Fri, Jan 23, 2009 at 7:51 AM, Ramon Buckland wrote: > Just had a thought. > > I will put the Generic* into camel-core.. and taylor each patch from there ? > How does that sound ? +1 that is fine as well Its new files into core and thus wont affect anything else. > > by the way. Having all of those test cases for the FTP was fantastic. It > makes me confident in the change as 80% of the tests failed, then minor > fixes and I had it all working again. So good work to all the test makers. Well it have taken quite some time to add all these tests. But yeah its important that we have so many for the file/ftp components. > > On Fri, Jan 23, 2009 at 17:33, Ramon Buckland wrote: > >> > Ramon, what do you say? >> >> Sounds Good. >> >> I will start with (1) very shortly (about to hop on ferry for home). >> >> r. >> > -- Claus Ibsen Apache Camel Committer Open Source Integration: http://fusesource.com Blog: http://davsclaus.blogspot.com/
Re: [jira] Updated: (CAMEL-1241) Add camel-commons-vfs component
Just had a thought. I will put the Generic* into camel-core.. and taylor each patch from there ? How does that sound ? by the way. Having all of those test cases for the FTP was fantastic. It makes me confident in the change as 80% of the tests failed, then minor fixes and I had it all working again. So good work to all the test makers. On Fri, Jan 23, 2009 at 17:33, Ramon Buckland wrote: > > Ramon, what do you say? > > Sounds Good. > > I will start with (1) very shortly (about to hop on ferry for home). > > r. >
Re: [jira] Updated: (CAMEL-1241) Add camel-commons-vfs component
> Ramon, what do you say? Sounds Good. I will start with (1) very shortly (about to hop on ferry for home). r.
Re: [jira] Updated: (CAMEL-1241) Add camel-commons-vfs component
Hi Ramon Yeah its a big refactor. Looks good. I do think we should consider java generics (the stuff) for the Generic* interfaces, but that is something we can get done later. Looks great and impressive that all the unit tests still passes. Since its a big refactor and the diff is huge we should avoid commiting to FTP/File in the trunk right now. To avoid the diff getting out of hands. Well this is for the Camel committers! To get the stuff into the trunk I think we need multiple diffs to avoid one huge. So if you can provide diffs in multiple steps. A rough plan like this: 1) Generic* that are used by the FTP component 2) When #1 is commited to triunk and you code is aligned with trunk, then move on to 3) Move Generic* from camel-ftp to camel-core 4) When #3 is commited to triunk and you code is aligned with trunk, then move on to 5) You can start on commons-vfs component 6) Then we can take a look at the java generics stuff and get that done in the right way 7) Refactor File in camel-core to use the new generics 8) Add the new commons-vfs component when its ready and done 9) Add wiki doc for the new commons-vfs component Ramon, what do you say? On Fri, Jan 23, 2009 at 7:14 AM, Ramon Buckland wrote: > Claus (and others) > > If you have time, can you please take a cursory glance at my refactoring. I > think there are a few rough edges which I am hoping to sort out when I > implement the commons-vfs from the Generic* > > The crux of the change so far is as follows. > > (a) refactor org.apache.camel.component.file.remote.Remote* so there is a > base that can be extended. > > * All current FTP Remote code resides in > org.apache.camel.component.file.remote > * It's parent Abstract base, interface and superclasses are > org.apache.camel.component.file.Generic* > * FTP and SFTP are now concrete implementations (no sharing of protocol in > same class); common code is pushed to Remote* classes. > > The next part will be to > > (b) Implement commons-vfs undeneath org.apache.camel.component.file.Generic* > (c) move org.apache.camel.component.file.Generic* to camel-core. > (d) Have camel-core:file implement / extend > org.apache.camel.component.file.Generic* > -- Claus Ibsen Apache Camel Committer Open Source Integration: http://fusesource.com Blog: http://davsclaus.blogspot.com/
Re: [jira] Updated: (CAMEL-1241) Add camel-commons-vfs component
Claus (and others) If you have time, can you please take a cursory glance at my refactoring. I think there are a few rough edges which I am hoping to sort out when I implement the commons-vfs from the Generic* The crux of the change so far is as follows. (a) refactor org.apache.camel.component.file.remote.Remote* so there is a base that can be extended. * All current FTP Remote code resides in org.apache.camel.component.file.remote * It's parent Abstract base, interface and superclasses are org.apache.camel.component.file.Generic* * FTP and SFTP are now concrete implementations (no sharing of protocol in same class); common code is pushed to Remote* classes. The next part will be to (b) Implement commons-vfs undeneath org.apache.camel.component.file.Generic* (c) move org.apache.camel.component.file.Generic* to camel-core. (d) Have camel-core:file implement / extend org.apache.camel.component.file.Generic*
[jira] Updated: (CAMEL-1241) Add camel-commons-vfs component
[ https://issues.apache.org/activemq/browse/CAMEL-1241?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ramon Buckland updated CAMEL-1241: -- Attachment: commons-vfs-changes-part1.file-refactoring.diff Attached is the refactoring to "extract out" the "generic File stuff" from the RemoteFile* implementation. All the tests are working as they were with this refactoring. Now I will create a commons-vfs implementation underneath the Generic* Note, I am uploading this now so that other can see what I am currently doing, get an idea for it. Hopefully I will have commons-vfs completed by tomorrow, at which time I will supply the new patch that is EVERYTHING including this attached one. > Add camel-commons-vfs component > --- > > Key: CAMEL-1241 > URL: https://issues.apache.org/activemq/browse/CAMEL-1241 > Project: Apache Camel > Issue Type: New Feature >Affects Versions: 2.0.0 >Reporter: Ramon Buckland >Assignee: Ramon Buckland > Fix For: 2.0.0 > > Attachments: commons-vfs-changes-part1.file-refactoring.diff > > > Add a new component camel-commons-vfs which utilises the commons-vfs library > providing access to zip,jar,tgz,cifs,ftp and many other wonderful "file" type > resources. > The component will go into camel 2.0 as the new camel-file and camel-ftp > components have been re-written to support a better model. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
