Fix externalSettings race

Created by: ChrisSchinnerl

Increment the revision counter inside of externalSettings causes a race since the caller doesn't always hold a write lock when calling externalSettings. Our convention dictates that an unexported method has to be called under a write lock though.