----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25435/#review52591 -----------------------------------------------------------
Thanks for this minor features, I can think of many people using it. Please see my review. api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java <https://reviews.apache.org/r/25435/#comment91440> Does this cause any issue? I would avoid this though there is no much different, long will occupy some more bytes than Int, but it would require us to note change in the API. api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java <https://reviews.apache.org/r/25435/#comment91441> If this is going to be UUID, please use the CommandType.UUID? server/src/com/cloud/usage/UsageServiceImpl.java <https://reviews.apache.org/r/25435/#comment91442> Avoid wildcard imports, dao.* here. server/src/com/cloud/usage/UsageServiceImpl.java <https://reviews.apache.org/r/25435/#comment91443> have all the case block in separate lines, improves readability. like: switch (var) { case 1: case 2: case 3: <do you stuff> <break> ... etc } - Rohit Yadav On Sept. 8, 2014, 1:45 p.m., Ilia Shakitko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25435/ > ----------------------------------------------------------- > > (Updated Sept. 8, 2014, 1:45 p.m.) > > > Review request for cloudstack, Alena Prokharchyk, Kishan Kavala, and Sheng > Yang. > > > Bugs: CLOUDSTACK-7159 > https://issues.apache.org/jira/browse/CLOUDSTACK-7159 > > > Repository: cloudstack-git > > > Description > ------- > > Working with Usage server and usage records very often I need to get only > records for that particular usage ID. For example when filtering out > network_bytes_received/sent with big amount of data it's not very fast to > process hundreds of objects looking for the only one you need. > It would be useful to have an ability to filter out usage records only for > specific resource ID. > > This parch brings that to the API. > > > Diffs > ----- > > api/src/org/apache/cloudstack/api/ApiConstants.java 6baa95c > > api/src/org/apache/cloudstack/api/command/admin/usage/GetUsageRecordsCmd.java > 21a7e4a > server/src/com/cloud/usage/UsageServiceImpl.java d1f62aa > > Diff: https://reviews.apache.org/r/25435/diff/ > > > Testing > ------- > > Tested cases: > - usageid is specified w/o "type": an exception is thrown (correct) > - provided usageid is not exists: an empty response is returned (since no > records were found, correct) > - no usageid specified: work as is > - an existing usageid specified (with type, for example type=4 or type=5): > only records for that usage type is returned > > > Thanks, > > Ilia Shakitko > >