170 похожих чатов

Коллеги, добрый день! Подскажите, пожалуйста, по Mutex: у меня есть такая структура: type

Service struct {
dao *dao.Dao
mu *sync.Mutex
ctx context.Context
}

У структура есть метод, который вызывается при работе 10+ горутин:
func (s *Service) getMetadataInstant(spec int, hash string) *metadata.Instant {
s.mu.Lock()
defer s.mu.Unlock()
metadataInstant, ok := metadata.RuntimeMetadata[spec]
if !ok {
raw := s.dao.RuntimeVersionRaw(spec)
if raw.Raw == "" {
raw.Raw = s.regCodecMetadata(hash)
}
metadataInstant = metadata.Process(raw) //тут race condition,
// metadata.Process(raw) пишет в глобальную мапу в пакете metadata
}
return metadataInstant
}
Не смотря на то, что я блокирую мютекс, я получил в коде race condition. Такого быть не должно, если мютекс заблокирован, или я неправильно понимаю суть мютекса?

7 ответов

17 просмотров

Возможно у вас в другом месте идет работа с этой же map, либо у вас есть копия этого сервиса

возможно вам лок нужно перенести в место где map меняется

Alexqrid- Автор вопроса
Roman 🎄
Возможно у вас в другом месте идет работа с этой ж...

Спасибо, но в коде 1 сервис только. и он запускает пул горутин с помощью panjf2000/ants/v2, которые работают и в какой-то момент дёргают эту функцию, но копия сервиса одна.

Alexqrid- Автор вопроса
Igor Kuznetsov
возможно вам лок нужно перенести в место где map м...

Это конечно идеальный вариант, но там либа сторонняя, которая постоянно обновляется, можно конечно PR послать, наверное попробую, кстати, спасибо)

> metadata.Process(raw) пишет в глобальную мапу в пакете metadata а оно синхронизировано?

Alexqrid- Автор вопроса
c
> metadata.Process(raw) пишет в глобальную мапу в ...

но я для этого мютекс использую, чтобы именно этот участок,где происходит запись, заблочить

Alexqrid
Это конечно идеальный вариант, но там либа сторонн...

еще вариант что вывоз функции process можно делать не в каждой горутине а только в одной, а остальные е б ей передавали данные через канал

Похожие вопросы

Обсуждают сегодня

Всем привет! Имеется функция: function IsValidChar(ch: UTF8Char): Boolean; var i: Integer; ValidChars: AnsiString; begin ValidChars := 'abcdefghijklmnopqrstuvwxyzABCDE...
Евгений
44
И никого не интересует какие пакеты кто использует. ((% Заходишь на сайт симфони и видишь поддержку Украины - по законам РФ это ж экстремизм. Только никто не отказывается от с...
Am Ambrion
11
лучше скажите, причём тут паскаль?
Alexey Kulakov
36
Чтобы перехватить все нажимания буков на форме, надо хук ставить? Пробовал на форме ОнКейДаун, оно ловит клаву если фокус не на компоненте с вводом текста
Serjone
15
Но, может, есть уже проверенная? Наши требования такие: 1. Сообщения должны приходить из Инста в CRM оду 2. Должна быть возможность подключить несколько экаунтов Инстаграм. Р...
Alexander Sharoiko MSE / Александр Шаройко
7
Народ! Впервые клиенту пришло письмо от РКН, у вас, дескать, есть яндекс метрика, а нигде не написано, что вы ее юзаете. Никто не сталкивался?
Sasha Beep
14
Всем привет! вывожу на общей стр дочерние ресурсыв каждом ресурсе галерея, и первая фотка должна выводиться на общей [!DocLister? &prepare=photo !]
Alekso
12
Я правильно понимаю что нет способов получить список ожидающих заявок на вступление в группу с помощью бота из mtproto?
Шамиль Прилов
7
А можно вопрос? Мне сегодня сказали что у меня функция (которая просто заполняет массив значениями) не правильная void Full(double * arr, int n) { for (int i = 0; i < n; i...
† C E †
7
Добрый вечер. Хочу чтобы у меня в классе поле было функцией, которая возвращает строку. Делаю так: interface ... TGetOutPath = function : String of object; ... protec...
Kirill Filippenok
12
Карта сайта