Hi Shakila,

I have reviewed your fix.

Basically their requirement is to create a folder for the date to move the
processed file.

I think we can do the same by using following parameter combination. WDYT?
We may not need to patch it?

   1. transport.vfs.SubFolderTimestampFormat
   2. transport.vfs.CreateFolder

Also observed that the PR you send does not contain the latest from develop
branch. If we merge this we may loose some commits already done.
eg:-
PR says,
*-file.optional=20*
+file.optional=23

But actually in develop branch,
*file.optional=22*

On Mon, May 18, 2015 at 12:47 PM, Shakila Sivagnanarajah <[email protected]>
wrote:

> Hi,
>
> I added the feature mentioned in ESBJAVA-2509 and added one missing vfs
> parameter. Please find the pull request at [1].
>
> [1] https://github.com/wso2/carbon-mediation/pull/159/
>
> Thank you
> --
> Shakila Sivagnanarajah
> Associate Software Engineer
> Mobile :+94 (0) 770 760240
> [email protected]
>



-- 

Best Regards,

Malaka Silva
Senior Tech Lead
M: +94 777 219 791
Tel : 94 11 214 5345
Fax :94 11 2145300
Skype : malaka.sampath.silva
LinkedIn : http://www.linkedin.com/pub/malaka-silva/6/33/77
Blog : http://mrmalakasilva.blogspot.com/

WSO2, Inc.
lean . enterprise . middleware
http://www.wso2.com/
http://www.wso2.com/about/team/malaka-silva/
<http://wso2.com/about/team/malaka-silva/>

Save a tree -Conserve nature & Save the world for your future. Print this
email only if it is absolutely necessary.
_______________________________________________
Dev mailing list
[email protected]
http://wso2.org/cgi-bin/mailman/listinfo/dev

Reply via email to