---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16813/#review31950
---
Ship it!
Applied to MASTER.
Thanks.
- Sheng Yang
On Jan. 15, 20
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16813/
---
(Updated Jan. 15, 2014, 7:23 p.m.)
Review request for cloudstack.
Changes
---
That's not how we deal with patches...
Please merge them into one patch and upload to the reviewboard as diff v2
instead. That would be easier for review.
Thanks.
--Sheng
On Tue, Jan 14, 2014 at 5:36 PM, Suresh Balineni <
sbalin...@service.juniper.net> wrote:
> Hi Sheng,
>
> This patch cont
Hi Sheng,
This patch contains two commits..
1. Previous patch.
2. New updated code with DB transactions removed.
If look at the second commit, you will see added diff.
Thanks,
Suresh
Sent from my iPhone
On Jan 14, 2014, at 5:32 PM, "Sheng Yang"
mailto:sh...@yasker.org>> wrote:
This is an aut
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16813/#review31839
---
I didn't see the updated patch. Both attached file and diff are old
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16813/
---
(Updated Jan. 14, 2014, 10:56 p.m.)
Review request for cloudstack.
Changes
--
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16813/#review31800
---
Dao.update() doesn't need to be wrapped by transcation.
- Sheng Ya
mailto:sbalin...@juniper.net>>
Reply-To: Suresh Balineni
mailto:sbalin...@service.juniper.net>>
Date: Sunday, January 12, 2014 5:02 PM
To: Suresh Balineni
mailto:sbalin...@service.juniper.net>>,
cloudstack mailto:dev@cloudstack.apache.org>>
Subject: Review Request 16813: Contrail
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16813/
---
Review request for cloudstack.
Repository: cloudstack-git
Description
---
