[ 
https://issues.apache.org/jira/browse/HADOOP-10201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13863322#comment-13863322
 ] 

Owen O'Malley commented on HADOOP-10201:
----------------------------------------

A few more points:
* Change the test case to check that the right number of elements in are each 
list. It would be even better to check the contents of the list.
* I think the loop for getKeyVersions would look better like:
{code}
       for (int i = 0; i < latestVersion; i++) {
         versionName = buildVersionName(name, i);
         KeyVersion v = getKeyVersion(versionName);
         if (v != null) {
            list.add(v);
         }
       }
{code}

> Add Listing Support to Key Management APIs
> ------------------------------------------
>
>                 Key: HADOOP-10201
>                 URL: https://issues.apache.org/jira/browse/HADOOP-10201
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: security
>            Reporter: Larry McCay
>            Assignee: Larry McCay
>         Attachments: 10201-2.patch, 10201-3.patch, 10201-4.patch, 10201.patch
>
>
> Extend the key management APIs from HADOOP-10141 to include the ability to 
> list the available keys.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to