Hello Daan,
Can you please cherry-pick the following commit from 4.4-forward to 4.4 ?
commit 48ea9e0b5e87fee067b711890cd5a5d7c9079bf1
CLOUDSTACK-6599:
1. Adding the missing Template/Volume URLs expiration functionality
2. Improvement - While deleting the volume during expiration use r
That's a lot of improvements without tests, Nitin.
On Fri, May 30, 2014 at 8:14 PM, Nitin Mehta wrote:
> Hello Daan,
>
> Can you please cherry-pick the following commit from 4.4-forward to 4.4 ?
>
> commit 48ea9e0b5e87fee067b711890cd5a5d7c9079bf1
> CLOUDSTACK-6599:
> 1. Adding the missing
Daan - Here improvements are actually bug fixes that should be fixed.
Thanks,
-Nitin
On 30/05/14 1:47 PM, "Daan Hoogland" wrote:
>That's a lot of improvements without tests, Nitin.
>
>On Fri, May 30, 2014 at 8:14 PM, Nitin Mehta
>wrote:
>> Hello Daan,
>>
>> Can you please cherry-pick the follo
Please cherry-pick be765ce8680564b743a73dd360c590c0e495c204 as well as
part of this bug.
One more thing to add, majority of code is for the functionality which I
found missing in 4.4 and found some bugs which I termed as improvements
over previous design.
Thanks,
-Nitin
On 30/05/14 3:06 PM, "Niti
now your are changing schema in java code! please don't do that. Those
are for data migration. If we start 5.0 we want to be able read the
sql to find the actual schema.
http://stackoverflow.com/questions/972922/add-column-to-mysql-table-if-it-does-not-exist
http://stackoverflow.com/questions/1438
Daan - Thanks for your comment. The java upgrade code has never been data
migration code only.
I have always seen complex DDL logic being handled by java upgrade code
because we can write complex logic and catch exceptions gracefully. As you
see from the links below - add column if it doesn't exist
strong point,
I added both commits. Are there tests to the old code as well? this is
a lot of untested code like this.
On Mon, Jun 2, 2014 at 7:50 PM, Nitin Mehta wrote:
> Daan - Thanks for your comment. The java upgrade code has never been data
> migration code only.
> I have always seen comple
Thanks Daan. Regarding test cases, I will check and get back.
Just that we don¹t loose this it in mails I have created a bug for 4.4,
for adding test cases if they don¹t already exist.
-Nitin
On 03/06/14 1:08 AM, "Daan Hoogland" wrote:
>strong point,
>
>I added both commits. Are there tests to