Hi Shakila,

Yes Please do so.

So basically your fix should only include transport.vfs.MoveTimestampFormat
parameter?

I guess they are referring to axis2 VFS in the jira. We may need to do the
same for both. (Inbound and VFS)

On Tue, May 19, 2015 at 9:28 AM, Shakila Sivagnanarajah <[email protected]>
wrote:

> Hi,
>
> Yes I have done that with transport.vfs.SubFolderTimestampFormat,
> transport.vfs.CreateFolder parameters only. And added one another
> parameter. But the feature that already written will create only one
> sub-folder with date. But the expected feature from the JIRA is creating
> sub-folders with year, month and date like,
>
> 2013
> 09
> 29
> File1
> File2
> File3
> 30
> File4
> File5
> File6
>
> So I moderated that accordingly. Can I give the PR again?
>
> Thank you
>
>
>
> On Tue, May 19, 2015 at 9:06 AM, Malaka Silva <[email protected]> wrote:
>
>> 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.
>>
>
>
>
> --
> 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