with these details.
Likitha,
Will be ignoring this patch as per the review discussion.
-Prachi
>-Original Message-
>From: Nitin Mehta [mailto:nore...@reviews.apache.org] On Behalf Of
>Nitin Mehta
>Sent: Friday, July 06, 2012 4:52 PM
>To: cloudstack; Prachi Damle; Likitha Sh
the review discussion.
-Prachi
>-Original Message-
>From: Nitin Mehta [mailto:nore...@reviews.apache.org] On Behalf Of
>Nitin Mehta
>Sent: Friday, July 06, 2012 4:52 PM
>To: cloudstack; Prachi Damle; Likitha Shetty; Nitin Mehta
>Subject: Re: Review Request: listSnapshotPolicies com
..@reviews.apache.org] On Behalf Of Nitin
>Mehta
>Sent: Friday, July 06, 2012 4:52 PM
>To: cloudstack; Prachi Damle; Likitha Shetty; Nitin Mehta
>Subject: Re: Review Request: listSnapshotPolicies command, volumeId
>parameter made optional.
>
>
>
>> On June 28, 2012, 1
Mehta
Subject: Re: Review Request: listSnapshotPolicies command, volumeId parameter
made optional.
> On June 28, 2012, 12:21 p.m., Nitin Mehta wrote:
> > Ideally listSnapshotPolicy should be part of
> > BaseListProjectAndAccountResourcesCmd.
>
> Prachi Damle wrote:
> On June 28, 2012, 12:21 p.m., Nitin Mehta wrote:
> > Ideally listSnapshotPolicy should be part of
> > BaseListProjectAndAccountResourcesCmd.
>
> Prachi Damle wrote:
> listSnapshotPolicy cannot extend BaseListProjectAndAccountResourcesCmd,
> since the snapshotPolicy entity does not carry
> On June 28, 2012, 12:21 p.m., Nitin Mehta wrote:
> > Ideally listSnapshotPolicy should be part of
> > BaseListProjectAndAccountResourcesCmd.
listSnapshotPolicy cannot extend BaseListProjectAndAccountResourcesCmd, since
the snapshotPolicy entity does not carry the account/domain attributes. I
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5604/#review8706
---
Ideally listSnapshotPolicy should be part of
BaseListProjectAndAccoun
