Task 06 - Parallel, Федоров Андрей, 393 группа#385
Conversation
AndreosGhost
commented
Nov 21, 2014
AndreosGhost
commented
There was a problem hiding this comment.
Это нормально, что size() выполняется не пойми сколько времени?
There was a problem hiding this comment.
Смотря на что расчет. Поддержка актуального значения size это, безусловно круто, но: приходится неявно немного дублировать логику с добавлениями/удалениями записей. Я под size подразумеваю следующее: кол-во записей, если вот-прям-сейчас сделать коммит. Что бы мы не подсчитывали для локального diff, другой поток может сделать коммит и локальный diff старого потока будет уже по-другому влиять на записи. В таком случае вычислять быстро size не предвидится возможным.
|
Validity control исправлен, полностью переписан и претворен в жизнь в следующем пулл-реквесте: #470 |
There was a problem hiding this comment.
Здесь точно нужен лок? Что сломается, если мы здесь уберём лок, и в параллель с этим методом другой поток будет что-то делать?
There was a problem hiding this comment.
Согласен, блокировка не нужна здесь. Починю в более новом задании)
|
#562 |
There was a problem hiding this comment.
Вообще, честно поддержать контракт по спецификации кажется достаточно сложным:
http://docs.oracle.com/javase/7/docs/api/java/util/Map.html#keySet()
"so changes to the map are reflected in the set, and vice-versa"
Так что и я не требую :)
Task 06 - Parallel, Федоров Андрей, 393 группа
3 participants
