[#CARROT-690] Unsynchronized access to Document fields via get(). - Carrot2 Clustering Engine Issues

Details

Type: Bug
Status: Resolved
Priority: Major
Resolution: Fixed
Affects Version/s: None
Fix Version/s: 3.4.0
Component/s: Framework Core
Labels:
None

Activity

Ascending order - Click to sort in descending order

Hide

Permalink

Dawid Weiss added a comment - 02/Jul/10 12:42 PM - edited

"I'd consider dropping these monitors, they are meaningless in all of our use case contexts."; [later] Not true, serialization can happen along with document modification; if so, the lock in there makes sense. What doesn't make sense is that serialization and document modification can happen alongside each other...

Show

Dawid Weiss added a comment - 02/Jul/10 12:42 PM - edited "I'd consider dropping these monitors, they are meaningless in all of our use case contexts."; [later] Not true, serialization can happen along with document modification; if so, the lock in there makes sense. What doesn't make sense is that serialization and document modification can happen alongside each other...

Hide

Permalink

Dawid Weiss added a comment - 05/Jul/10 10:57 AM

Fixed in trunk (added monitor to get()), but the fact that Document is mutable makes everything so much more complex than it should be.

Show

Dawid Weiss added a comment - 05/Jul/10 10:57 AM Fixed in trunk (added monitor to get()), but the fact that Document is mutable makes everything so much more complex than it should be.

People

Assignee:

Stanisław Osiński

Reporter:

Dawid Weiss

Vote (0)

Watch (0)

Dates

Created:

02/Jul/10 12:37 PM

Updated:

05/Jul/10 10:57 AM

Resolved:

05/Jul/10 10:57 AM