Move DescriptorNamespace out of the Public API ---------------------------------------------- Key: SHRINKDESC-113 URL: https://issues.jboss.org/browse/SHRINKDESC-113 Project: ShrinkWrap Descriptors Issue Type: Task Reporter: Andrew Rubinger Assignee: Michal Matloka This needs some discussion before being done. It'd break backwards-compatibility with the 1.x series. DescriptorNamespace is mutable, which conflicts with the goals outlined in SHRINKDESC-21. We're not really testing its direct usage at the moment, and I wonder what reason end-users would have to be invoking its operations to muck around with the "namespace" of a domain object. Currently all descriptors are supporting its operations by extension: {code}public interface WebAppDescriptor extends Descriptor, DescriptorNamespace<WebAppDescriptor>{code} ...and the methods in question as defined by DescriptorNamespace are: {code} public T addDefaultNamespaces(); public T addNamespace(String name, String value); public List<String> getNamespaces(); public T removeAllNamespaces();{code} Note that Ralf's work in SHRINKDESC-21 also splits this into DescriptorNamespace and DescriptorMutableNamespace. My feeling is that neither should be part of the API; maybe SPI or just internals. Discuss if it's appropriate to remove this and break the backwards-compatibility with the 1.x series. In this case, I'm willing to entertain this exception.