Seems like we probably don't need to inherit from BaseXMLImporter - unless we're going to implement the SAX reading