RFR (S) CR 8014966: Add the proper Javadoc to @Contended
David Holmes
david.holmes at oracle.com
Tue May 28 00:11:30 UTC 2013
Hi Aleksey,
Okay I have some suggested rewordings below and a couple of grammatical
corrections.
I've trimmed out parts that are unchanged.
On 27/05/2013 10:44 PM, Aleksey Shipilev wrote:
> A {@code @Contended} field annotation may optionally include a
> contention group tag. All fields with the same tag are considered as a
> group with respect to isolation from other groups. A default tag
> indicates the <i>distinct</i> and <i>anonymous</i> contention group,
> which implies the contention with all other fields, including all other
> {@code @Contended} ones. The tag at a class level is meaningless, and
> ignored.
A {@code @Contended} field annotation may optionally include a
<i>contention group</i> tag. A contention group defines a set of one or
more fields that collectively must be isolated from all other contention
groups. With no contention group tag (or with the default empty tag: "")
each {@code @Contended} field resides in its own <i>distinct</i> and
<i>anonymous</i> contention group. A contention group tag has no meaning
in a class level {@code @Contended} annotation, and is ignored.
> ... With the class level annotation, the implementations may
> choose different approach to protect the entire object, rather
> than protecting only the distinct fields.
With the class level annotation, implementations may choose different
isolation techniques, such as isolating the entire object, rather than
isolating distinct fields.
> The {@code @Contended} annotation is not inherited, and acts only
> within the class it was used on. The effects, however, should be
> enforced across the inheritance, e.g. the {@code @Contended} fields
> should be isolated from all the fields in super- and sub-classes. The
> contention group tags are not inherited as well: the similar tags in two
> subclasses mean different groups.
The class level {@code @Contended} annotation is not inherited and has
no effect on the fields of any sub-classes. The effects of all {@code
@Contended} annotations, however, remain in force for all subclass
instances, providing isolation of all the defined contention groups.
Contention group tags are not inherited, and the same tag used in a
superclass and subclass, represent distinct contention groups.
> * The (optional) contention group tag.
> * This tag is only meaningful for the field-level annotations.
Delete "the" in "for the".
Check "field-level" vs. "field level" usage; and "class-level" vs "class
level" usage. I'm not sure what the JDK conventions are so you may need
to find other examples. Just be sure to be internally consistent if
there is no clear convention.
Thanks,
David
------
More information about the core-libs-dev
mailing list