Code for review #3
Conversation
get_data.go
Outdated
| @@ -15,53 +16,34 @@ var ( | |||
| const timeout = time.Second | |||
|
|
|||
| func getter(key string) (any, error) { | |||
| // time.Sleep(1 * time.Microsecond) | |||
| // time.Sleep(2 * time.Second) | |||
There was a problem hiding this comment.
Так это намеренное замедление функции, чтобы по логике кэш получать
There was a problem hiding this comment.
Сейчас это мертвый код в комментариях)
| return nil, err | ||
| default: | ||
| } | ||
| select { |
There was a problem hiding this comment.
Это неправильно, у тебя же выше блокирующие вызовы на чтение из БД и кэша, поэтому никакого таймаута не будет
There was a problem hiding this comment.
Так это синхронное получение данных из источников, предполагается, что в select будет обрабатываться timeout. А как правильно должно быть?
There was a problem hiding this comment.
Словами не объяснишь - на кажжый синхронный вызов должна быть запущена горутина по очереди, а не параллельно и главное уметь ее поутшить при пробивании таймаута
There was a problem hiding this comment.
Понимаю к чему клонишь, но можешь примерно кодом набросать, чтобы было единое виденье
// GetData возвращает данные из getter, страхуя кешом
// в случае ошибки. Проблема: getter может отвечать
// очень долго.
// Задача: в случае ответа дольше timeout отдавать кеш,
// и правильно обработать кейс, когда getter всегда отвечает долго.