ok no strong opinion.
But you don't have to assume we expect all users to implement it; in practice this SPI is only expected to be implemented by myself in the scope of Infinispan, and I'd use the same ClassLoaderHelper as an helper; I just feel it makes for cleaner code as there seems to be quite some repetition. but ok