Skip to content

Add more detailed information about the compressed chunks#273

Merged
folbricht merged 1 commit intofolbricht:masterfrom
RyuzakiKK:precise_size
Nov 30, 2024
Merged

Add more detailed information about the compressed chunks#273
folbricht merged 1 commit intofolbricht:masterfrom
RyuzakiKK:precise_size

Conversation

@RyuzakiKK
Copy link
Copy Markdown
Contributor

list-chunks has now a new --detailed option that allows to print additional information about the chunks, including their compressed size (if a local store is provided).

One use case is for the clients to download the server's list-chunks JSON and feed it into desync info, to calculate the precise download size of an update.

Addresses #270

Comment thread cmd/desync/info.go Outdated
Comment thread cmd/desync/info.go
Comment thread cmd/desync/list.go Outdated
Comment thread cmd/desync/list.go Outdated
Comment thread cmd/desync/list.go Outdated
Comment thread local.go Outdated
@RyuzakiKK
Copy link
Copy Markdown
Contributor Author

Thanks for the review. By the end of the week I should be able to find some time to address the points raised.

@RyuzakiKK RyuzakiKK force-pushed the precise_size branch 2 times, most recently from b4e607b to 7acdd94 Compare November 14, 2024 16:53
Copy link
Copy Markdown
Owner

@folbricht folbricht left a comment

Choose a reason for hiding this comment

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

It's close, just left some minor comments

Comment thread cmd/desync/inspectchunks_test.go
Comment thread cmd/desync/info.go Outdated
Comment thread cmd/desync/inspectchunks.go Outdated
Comment thread cmd/desync/inspectchunks.go Outdated
Add a new `inspect-chunks` command that can provide additional
information about the chunks in an index, including their compressed
size (if a local store is provided).

One use case is for the clients to download the server's inspect-chunks
JSON and feed it into `desync info`, to calculate the precise download
size of an update.

Signed-off-by: Ludovico de Nittis <ludovico.denittis@collabora.com>
Copy link
Copy Markdown
Owner

@folbricht folbricht left a comment

Choose a reason for hiding this comment

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

Thank you. Would it be worth adding an example to the README for this?

@folbricht folbricht merged commit 0260be9 into folbricht:master Nov 30, 2024
JeremieA pushed a commit to InSimo/desync that referenced this pull request Apr 10, 2026
…icht#273)

Add a new `inspect-chunks` command that can provide additional
information about the chunks in an index, including their compressed
size (if a local store is provided).

One use case is for the clients to download the server's inspect-chunks
JSON and feed it into `desync info`, to calculate the precise download
size of an update.

Signed-off-by: Ludovico de Nittis <ludovico.denittis@collabora.com>
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.

2 participants