qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (QPID-5879) [Java Broker] Json Configuration Store should take deep copy of ConfiguredObjectRecords
Date Sun, 06 Jul 2014 23:10:34 GMT

    [ https://issues.apache.org/jira/browse/QPID-5879?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14053246#comment-14053246
] 

ASF subversion and git services commented on QPID-5879:
-------------------------------------------------------

Commit 1608311 from [~godfrer] in branch 'qpid/trunk'
[ https://svn.apache.org/r1608311 ]

QPID-5879 : [Java Broker] JsonConfigStore should take deep copy of ConfiguredObjectRecord
objects

> [Java Broker] Json Configuration Store should take deep copy of ConfiguredObjectRecords
> ---------------------------------------------------------------------------------------
>
>                 Key: QPID-5879
>                 URL: https://issues.apache.org/jira/browse/QPID-5879
>             Project: Qpid
>          Issue Type: Improvement
>          Components: Java Broker
>            Reporter: Rob Godfrey
>            Assignee: Rob Godfrey
>             Fix For: 0.29
>
>
> Currently the Json Config Store retains a reference to the ConfiguredObjectRecords passed
in in update and create operations.  If the ConfiguredObjectRecord is actually one generated
from AbstractConfiguredObject.asObjectRecord() then the records value may change, and querying
the attributes of the record will obtain locks on the underlying configured object.
> To prevent deadlocks and to ensure that the correct values get written to the store,
a deep copy of the record should be taken before storing.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

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


Mime
View raw message