Mailing-List: contact issues-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: issues@commons.apache.org
Date: Sat, 22 Oct 2016 20:04:58 +0000 (UTC)
From: "Gary Gregory (JIRA)" <jira@apache.org>
To: issues@commons.apache.org
Message-ID: <JIRA.12836937.1433966848000.60576.1477166698375@Atlassian.JIRA>
In-Reply-To: <JIRA.12836937.1433966848000@Atlassian.JIRA>
References: <JIRA.12836937.1433966848000@Atlassian.JIRA> <JIRA.12836937.1433966848319@arcas>
Subject: [jira] [Updated] (LANG-1144) Multiple calls of
 org.apache.commons.lang3.concurrent.LazyInitializer.initialize() are
 possible
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Sat, 22 Oct 2016 20:05:00 -0000


     [ https://issues.apache.org/jira/browse/LANG-1144?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gary Gregory updated LANG-1144:
-------------------------------
    Attachment:     (was: commons-lang.patch)

> Multiple calls of org.apache.commons.lang3.concurrent.LazyInitializer.initialize() are possible
> -----------------------------------------------------------------------------------------------
>
>                 Key: LANG-1144
>                 URL: https://issues.apache.org/jira/browse/LANG-1144
>             Project: Commons Lang
>          Issue Type: Bug
>          Components: lang.concurrent.*
>    Affects Versions: 3.4
>         Environment: Java 1.8 on Windows 7 x64
>            Reporter: Waldemar Maier
>            Assignee: Gary Gregory
>         Attachments: 0001-LANG-1144-allow-nulls-as-return-value.patch, commons-lang.patch
>
>
> It is possible to create a construct, that allows multiple calls of LazyInitializer.initialize, when calculations (which can be very expensive) return null as result. 
> In the Javadoc is described that the initialize method will be called only on the first access
> {code:java}
>     /**
>      * Creates and initializes the object managed by this {@code
>      * LazyInitializer}. This method is called by {@link #get()} when the object
>      * is accessed for the first time. An implementation can focus on the
>      * creation of the object. No synchronization is needed, as this is already
>      * handled by {@code get()}.
>      *
>      * @return the managed data object
>      * @throws ConcurrentException if an error occurs during object creation
>      */
>     protected abstract T initialize() throws ConcurrentException;
> {code}
> The Junit Test can be something like this:
> *(fix can be appplied from attached patch-file)*
> {code:java}
> package edu.test;
> import static org.junit.Assert.assertEquals;
> import org.apache.commons.lang3.concurrent.ConcurrentException;
> import org.apache.commons.lang3.concurrent.LazyInitializer;
> import org.junit.Test;
> public class LazyInitializerTest {
>   private int lazyinitCounter = 0;
>   private LazyInitializer<Object> lazyIinit = new LazyInitializer<Object>() {
>     @Override
>     protected Object initialize() throws ConcurrentException {
>       lazyinitCounter++;
>       return doSomeVeryExpensiveOperations();
>     }
>   };
>   
>   
>   private Object doSomeVeryExpensiveOperations() {
>     // do db calls
>     // do some complex math calculations
>     // the result of them all is null
>     return null;
>   }
>   
>   
>   @Test
>   public void testInitialization() throws Exception {
>     lazyIinit.get();
>     lazyIinit.get();
>     assertEquals("Multiple call of LazyInitializer#initialize", 1, lazyinitCounter);
>   }
> }
> {code}


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)