Skip to content

Optional to turn off chunk caching by Reductionist#321

Open
valeriupredoi wants to merge 5 commits intomainfrom
no_chunk_cache
Open

Optional to turn off chunk caching by Reductionist#321
valeriupredoi wants to merge 5 commits intomainfrom
no_chunk_cache

Conversation

@valeriupredoi
Copy link
Copy Markdown
Collaborator

@valeriupredoi valeriupredoi commented Mar 30, 2026

Description

As discussed with @maxstack and @varsiha-sothilingam this is the Client end of the implementation of a flag that turns off chunk caching (for tests, nightly tests, or other used cases when we don't want to write chunk caches). Max, at the moment I am getting this:

>           raise ReductionistError(response.status_code, error)
E           activestorage.reductionist.ReductionistError: Reductionist error: HTTP 400: {"error": {"message": "request data is not valid", "caused_by": ["Failed to deserialize the JSON body into the target type", "option_disable_chunk_cache: unknown field `option_disable_chunk_cache`, expected one of `interface_type`, `url`, `dtype`, `byte_order`, `offset`, `size`, `shape`, `axis`, `order`, `selection`, `compression`, `filters`, `missing` at line 1 column 426"]}}

I'd assume since the option is on a branch and not yet deployed on Reductionist v2? 🍻

Before you get started

Checklist

  • This pull request has a descriptive title and labels
  • This pull request has a minimal description (most was discussed in the issue, but a two-liner description is still desirable)
  • Unit tests have been added (if codecov test fails)
  • Any changed dependencies have been added or removed correctly (if need be)
  • If you are working on the documentation, please ensure the current build passes
  • All tests pass

@valeriupredoi valeriupredoi requested a review from maxstack March 30, 2026 13:30
@valeriupredoi valeriupredoi added enhancement New feature or request testing testing duh labels Mar 30, 2026
@valeriupredoi valeriupredoi changed the title No chunk cache optional Optional to turn off chunk caching by Reductionist Mar 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request testing testing duh

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant