The legacy variants of ManagementXml and AuditLogXml are getting loaded but aren't used, which is a waste of boot time and of metaspace.

It turns out that the idiom of having the impls extend an abstract base class and then having a static factory method in the abstract class results in all the different impl classes being loaded into memory. Simply changing the abstract base class to an interface eliminates the problem.

While I'm at it, the legacy impls still have code for the writeXXX methods, which should never be called. So I'll clean that out and replace with a default impl that throws UnsupportedOperationException.

I'll do some other code cleanup on these too, removing unused params, etc.

This should save 118K of metaspace.