[ 
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

Reply via email to