Yes, it should be synchronized, though it's harmless if it is initialized multiple times.
On Sat, Sep 8, 2012 at 8:51 AM, David Medinets <[email protected]>wrote: > Findbugs is warning (Incorrect lazy initialization and update of > static field) about the following code: > > public static SiteConfiguration getInstance(AccumuloConfiguration > parent) { > if (instance == null) { > instance = new SiteConfiguration(parent); > ConfigSanityCheck.validate(instance); > } > return instance; > } >
