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;
  }

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;
>   }
>