ravening commented on pull request #4634: URL: https://github.com/apache/cloudstack/pull/4634#issuecomment-900924374
> > > no comment on this PR but all this `DAO` code really shouldn't be in the `ManagementServer` > > > > > > @DaanHoogland any idea where else I can add this ? > > `DomainVlanMapDaoImpl`, i'd say. @DaanHoogland Im using the same code which is used by `account` and `pod` also which are present above and below of this code. So even they needs to be moved away? I searched in other places in the same file and they are also using the same logic. for eg: publicipaddress, loadbalancer, resourcetag, vlan, guestos -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
