Locking on a String is unsafe. String literals and interned strings are shared across the JVM, so two unrelated callers with the same definition name can accidentally contend on the same monitor.