JuliaIO / Zarr.jl / 27812659413

90%

push

github

fix: write chunks in the ConcurrentRead write_items! path (#297)

* fix: write chunks in the ConcurrentRead write_items! path

write_items!(::ConcurrentRead, ...) wrote a chunk with
`store_writechunk(s, data, p, ii, e) = data`, which Julia parses as a no-op
local function definition rather than a call. Stores using the ConcurrentRead
strategy (e.g. S3Store) therefore persisted array metadata but silently dropped
all chunk data, with no error raised. The SequentialRead path calls
store_writechunk correctly.

Drop the `= data` so the chunk is actually written. Adds a regression test
using a DictStore wrapper that advertises ConcurrentRead.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* docs: add changelog entry for #297

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

0 of 1 new or added line in 1 file covered. (0.0%)

1667 of 1862 relevant lines covered (89.53%)

18669.06 hits per line