commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Oliver Heger <oliver.he...@oliver-heger.de>
Subject Re: [configuration] svn commit: r548098 - in /jakarta/commons/proper/configuration/trunk: src/java/org/apache/commons/configuration/ xdocs/ xdocs/userguide/
Date Sun, 17 Jun 2007 20:26:55 GMT
Rahul Akolkar wrote:
> On 6/17/07, oheger@apache.org <oheger@apache.org> wrote:
>> Author: oheger
>> Date: Sun Jun 17 12:34:03 2007
>> New Revision: 548098
>>
>> URL: http://svn.apache.org/viewvc?view=rev&rev=548098
>> Log:
>> Javadoc only: added notes about thread-safety to the most important 
>> Configuration implementations
>>
> <snip/>
>> +
>> +      <subsection name="Threading issues">
>> +      <p>
>> +        The most concrete implementations of the 
>> <code>Configuration</code>
>> +        interface that are shipped with this library are thread-safe 
>> as long as
>> +        they are accessed in a read-only manner. However if one thread
>> +        modifies a configuration object, manual synchronization has 
>> to be
>> +        performed to ensure correctness of data. Notes about the thread
>> +        safety of conrete implementation classes can be found in the 
>> Javadocs
>> +        for these classes.
>> +      </p>
>> +      </subsection>
> <snap/>
> 
> I think the first sentence should simply state that most
> implementations are not thread-safe (rather than the read-only bit
> which I found unnecessary, perhaps confusing).
> 
> WDYT?
> 
> -Rahul
> 
How about:
"The most concrete implementations of the <code>Configuration</code> 
interface that are shipped with this library are not thread-safe. They 
can be accessed concurrently in a read-only manner. However if one 
thread modifies a configuration object, manual synchronization has to be 
        performed to ensure correctness of data."

Oliver

BTW: Thanks for the hint.

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message