[jira] [Resolved] (CLOUDSTACK-1080) Interface heirarchy for AccountManagerImpl superflous

["Rohit Yadav (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/CLOUDSTACK-1080?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Rohit Yadav resolved CLOUDSTACK-1080.
-------------------------------------

    Resolution: Invalid

The submitted patch is not acceptable, you removed the interfaces, class needs to implement those interfaces that's why "class implements interfaces".

Thanks for your submission, closing this issue as invalid. Pl. reopen if can explain what you're trying to do or if you've a new patch. If refactoring account manager and service interfaces together, open a new issue and new review on reviews.apache.org
                
> Interface heirarchy for AccountManagerImpl superflous
> -----------------------------------------------------
>
>                 Key: CLOUDSTACK-1080
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-1080
>             Project: CloudStack
>          Issue Type: Improvement
>      Security Level: Public(Anyone can view this level - this is the default.) 
>          Components: API
>            Reporter: Joe San
>            Assignee: Rohit Yadav
>            Priority: Trivial
>              Labels: Refactoring
>
> The AccountManagerImpl.java has two interfaces in its implements clause which is a bit redundant. These two interfaces are already in the same heirarchy (AccountManager and AccountService). Can we refactor the AccountManagerImpl to implement only the AccountManager?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira