My Review of sun-domain_1_4.dtd – Kedar

Item Comment Decision
0 It seems odd to me that sip-container is a separate node from sip-service. Are they semantically different?  
1 Remove all references to SE/EE from the DTD. Sorry, I left some behind.  
2 Converged-load-balancer@config-file: Is this is the converged-load-balancer.xml file? In other words, is this is the file that admin code is going to generate? If that's the case, why should this be configurable? Are we saying that administrator would want this file to be created by admin code in a specific place?? Please clarify.  
3 I have tough time understanding the need of the element converged-lb-config. All it has an attribute named reload-poll-interval-in-seconds besides "name" which is just for referencing. If that's the case, why not have the attribute along with the converged-load-balancer itself? Do we want to keep these two separate? I don't expect two converged-load-balancer elements to reference the same converged-lb-config element just to share one attribute.  
4 Why do we need a top-level converged-lb-configs element? Can't the converged-lb-config element reside inside lb-configs element? Ultimately, it is just the configuration of a load balancer, right?