Skip to content
This repository was archived by the owner on Apr 18, 2025. It is now read-only.

[FIX] Chunk builder's block limit #335

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

noel2004
Copy link
Member

When chunk builder deliver chunk data with block limit condition, the original code do not reset the ccc result.

As the result, the following add call would deliver a empty collection of chunks and panic.

This PR fix the issue and add enviroment variant (CHUNK_BLOCK_LIMIT) for setting the block limit of chunk builder.

For not-set, thhe block limit is kept as original code (1) and to enable unlimit of blocks, use CHUNK_BLOCK_LIMIT=0

@lispc
Copy link
Collaborator

lispc commented Aug 13, 2024

i guess you find this bug by set block_limit > 1?

@noel2004
Copy link
Member Author

i guess you find this bug by set block_limit > 1?

yes

@0xShankar
Copy link

Try Block_limit

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants