Skip to content

Further tests and delete chunk data after resize#49

Merged
brokkoli71 merged 11 commits into
mainfrom
further-tests
Feb 6, 2026
Merged

Further tests and delete chunk data after resize#49
brokkoli71 merged 11 commits into
mainfrom
further-tests

Conversation

@brokkoli71

@brokkoli71 brokkoli71 commented Jan 26, 2026

Copy link
Copy Markdown
Contributor
  • fixes [v2] Further Tests #20
  • adds deleting unused chunk data on Array.resize that can be disabled with resizeMetadataOnly argument.

# Conflicts:
#	src/test/java/dev/zarr/zarrjava/TestUtils.java
@brokkoli71
brokkoli71 marked this pull request as ready for review January 26, 2026 12:58
@brokkoli71
brokkoli71 requested a review from normanrz January 26, 2026 12:58

@normanrz normanrz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work!

Comment thread src/test/java/dev/zarr/zarrjava/ZarrV2Test.java
Comment thread src/test/java/dev/zarr/zarrjava/ZarrV3Test.java
@brokkoli71 brokkoli71 self-assigned this Jan 30, 2026
# Conflicts:
#	src/main/java/dev/zarr/zarrjava/v3/Group.java
@brokkoli71
brokkoli71 requested a review from normanrz January 30, 2026 11:05
@brokkoli71 brokkoli71 changed the title Further tests Further tests and delete chunk data after resize Jan 30, 2026
Comment thread src/main/java/dev/zarr/zarrjava/v2/Array.java Outdated
@brokkoli71
brokkoli71 merged commit 32f785f into main Feb 6, 2026
3 checks passed
@brokkoli71
brokkoli71 deleted the further-tests branch February 6, 2026 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[v2] Further Tests

2 participants