diff --git a/internal/storage/iotpg/iot_telemetry_store.go b/internal/storage/iotpg/iot_telemetry_store.go index 8f79b9c..e6a88f9 100644 --- a/internal/storage/iotpg/iot_telemetry_store.go +++ b/internal/storage/iotpg/iot_telemetry_store.go @@ -162,18 +162,20 @@ func (s *IoTPostgresStore) flushAll(ctx context.Context) error { for ns, rows := range batches { if err := s.flushTenant(ctx, ns, rows); err != nil { s.log.Error("iotpg: flush batch", "namespace", ns, "rows", len(rows), "err", err) - if firstErr == nil { - firstErr = err - } if isRetryableFlushErr(err) { + if firstErr == nil { + firstErr = err + } // Сеть/PG временно недоступен — возвращаем строки в буфер, повторим. s.mu.Lock() s.batches[ns] = append(s.batches[ns], rows...) s.mu.Unlock() } else { // Постоянная ошибка: валидные строки уже вставлены построчным - // fallback'ом (см. flushTenant), битые отброшены с логом. Повтор - // бессмыслен — в буфер НЕ возвращаем (иначе залипание namespace). + // fallback'ом (см. flushTenant), битые отброшены с логом. Ошибку + // НЕ возвращаем и строки в буфер НЕ возвращаем — данные обработаны, + // иначе consumer увидел бы ошибку и бесконечно гонял poison-батч + // через visibility timeout (ревью 🔴). s.log.Error("iotpg: permanent flush error (валидные строки — построчно, битые отброшены)", "namespace", ns, "rows", len(rows), "err", err) } }