You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by Rahul Akolkar <ra...@gmail.com> on 2007/06/17 22:19:01 UTC

Re: [configuration] svn commit: r548098 - in /jakarta/commons/proper/configuration/trunk: src/java/org/apache/commons/configuration/ xdocs/ xdocs/userguide/

On 6/17/07, oheger@apache.org <oh...@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

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


Re: [configuration] svn commit: r548098 - in /jakarta/commons/proper/configuration/trunk: src/java/org/apache/commons/configuration/ xdocs/ xdocs/userguide/

Posted by Rahul Akolkar <ra...@gmail.com>.
On 6/17/07, Oliver Heger <ol...@oliver-heger.de> wrote:
<snip/>
> 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."
>
<snap/>

Yup, thanks!

-Rahul


> 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


Re: [configuration] svn commit: r548098 - in /jakarta/commons/proper/configuration/trunk: src/java/org/apache/commons/configuration/ xdocs/ xdocs/userguide/

Posted by Oliver Heger <ol...@oliver-heger.de>.
Rahul Akolkar wrote:
> On 6/17/07, oheger@apache.org <oh...@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