Well, I would not say "all we need". Moving forward I think removing the old entry from META-INF/services/javax.persistence.spi.PersistenceProvider is clearly the right thing to do, for the reasons I outlined above.