Yes, the idea is that this extractor has a cohesive piece of discrete functionality that just deals with reg svc attribute extraction. Any other additional functionality that is needed would be due in another extractor impl. Hence the final here. I'd leave it alone. Remember, simpler, micro-services architecture like ;-)