I'd rather not make FileSystemDatasetRepository mutable, or unnecessarily expose the repo URI in the builder, so having a subclass seems OK. We already have a subclass, HCatalogDatasetRepository, for the managed case.