You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sis.apache.org by "Martin Desruisseaux (JIRA)" <ji...@apache.org> on 2013/04/20 19:23:15 UTC

[jira] [Closed] (SIS-85) Consider removing metadata synchronization

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

Martin Desruisseaux closed SIS-85.
----------------------------------

    Resolution: Fixed

Committed in the JDK7 branch in revisions 1470184 to 1470195 inclusive (3 successive commits).
Committed (merged) in the JDK6 branch at revision 1470197.
Committed (merged) on trunk at revision 1470200.

                
> Consider removing metadata synchronization
> ------------------------------------------
>
>                 Key: SIS-85
>                 URL: https://issues.apache.org/jira/browse/SIS-85
>             Project: Spatial Information Systems
>          Issue Type: Improvement
>          Components: Metadata
>            Reporter: Martin Desruisseaux
>            Assignee: Martin Desruisseaux
>             Fix For: 0.3
>
>
> All subclasses of {{ISOMetadata}} are currently synchronized. It probably doesn't have significant impact on the JAR file size, since (in my understanding) adding a {{synchronized}} keyword in the method signature just set a bit in a bitmask. However it may be a cause of thread contention.
> Synchronization may not be necessary since the typical usage of metadata objects is:
> * Create the metadata object once in a single thread.
> * Declare the metadata as unmodifiable.
> * Allow any thread to read (but not set) the metadata properties.
> The potential contention, especially on metadata object shared by many other objects, would be unfortunate. However profiling this not show significant bottleneck yet.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira