Mark McDonnell

So yeah at the moment I'm actually just getting by with `slog.NewLogLogger()` ```go redis.NewLoggingConn(c, slog.NewLogLogger(std.Handler(), slog.LevelInfo), "redis") ``` This is fine to unblock me, but I would imagine _actual_ support...

Hi @alwin-stripe Thanks for opening this issue. So a couple of things to mention... 1. It makes sense to support contexts (thank you for the suggestion). 2. If we did...

Great. Good to hear you have a workaround for the time being. I appreciate the update 👍🏻

👋🏻 Thanks @shavounet for opening this PR. You mentioned... > from what've tested, huge parallelized batches seem to fail with 429 errors (but smalls are ok) Have you tested with...

> It fails fast with 429 errors with too much parallelization OK, no problem. That seems fine to me. It's the caller's responsibility (not the API client's) to respect the...

Sorry, just saw your edit note. I think it's a general API rate limit and not product specific... https://www.fastly.com/documentation/reference/api/#rate-limiting

I'll look to get this merged later today and hopefully a new released published soon after.

> Do you know how the rate limit is triggered while batching ? (1 per API request, even if it contains thousands of lines, or 1 per row ?) Great...

Thanks @shavounet for your patience. We realise now there is a gap in our KV Store documentation, that we will fix (thanks for making us aware of this). In short,...

I'm just testing this PR locally by integrating it into the [Fastly CLI](https://github.com/fastly/cli), if all is well I'll merge this PR.