Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

thread 'solana-record-stage' panicked at 'assertion failed: serialized_size(&entry).unwrap() <= BLOB_DATA_SIZE as u64', src/entry.rs:67:9 #740

Closed
mvines opened this issue Jul 24, 2018 · 6 comments · Fixed by #751
Closed

thread 'solana-record-stage' panicked at 'assertion failed: serialized_size(&entry).unwrap() <= BLOB_DATA_SIZE as u64', src/entry.rs:67:9 #740

mvines opened this issue Jul 24, 2018 · 6 comments · Fixed by #751
Assignees
@mvines
Milestone
v0.8 Windansea

Comments

@mvines
Copy link
Contributor

mvines commented Jul 24, 2018

This assert is failing on the testnet leader sometimes:
https://github.com/solana-labs/solana/blob/73ae3c3301805/src/entry.rs#L67

thread 'solana-record-stage' panicked at 'assertion failed: serialized_size(&entry).unwrap() <= BLOB_DATA_SIZE as u64', src/entry.rs:67:9
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
stack backtrace:
   ...
   7: solana::entry::Entry::new
   8: solana::ledger::next_entries_mut
   9: solana::record_stage::RecordStage::process_signal
@mvines mvines added this to the v0.8.0 milestone Jul 24, 2018
@mvines mvines self-assigned this Jul 24, 2018
@mvines mvines mentioned this issue Jul 24, 2018
Merged
@mvines
Copy link
Contributor Author

mvines commented Jul 24, 2018

New info from #774:

thread 'solana-record-stage' panicked at 'Serialized entry size too large: 66204 (292 transactions):', src/entry.rs:70:13

@rob-solana
Copy link
Contributor

lots of votes, I guess!

@mvines
Copy link
Contributor Author

mvines commented Jul 24, 2018

The transactions should be getting chunked well before 292 though! https://github.com/solana-labs/solana/blob/master/src/ledger.rs#L77-L79

@rob-solana
Copy link
Contributor

yep, I don't see how 292 gets through unless that vector of transactions is being modified while being passed to recorder

@mvines
Copy link
Contributor Author

mvines commented Jul 24, 2018

My current guess is that chunk_len = 1 here [1].

[1] https://github.com/solana-labs/solana/blob/master/src/ledger.rs#L82

@rob-solana
Copy link
Contributor

I don't get it. chunk_len = 1 would mean a lot of single-Transaction Entries, right?

@mvines mvines mentioned this issue Jul 25, 2018
Closed
@rob-solana rob-solana mentioned this issue Jul 25, 2018
Merged
vkomenda pushed a commit to vkomenda/solana that referenced this issue Aug 29, 2021
@dependabot
Bump eslint from 7.12.0 to 7.12.1 in /token-swap/js (solana-labs#740)
46500ad 
Bumps [eslint](https://github.com/eslint/eslint) from 7.12.0 to 7.12.1.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](eslint/eslint@v7.12.0...v7.12.1)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@mvines mvines

Labels
None yet
Projects
None yet
Milestone
v0.8 Windansea
Development

Successfully merging a pull request may close this issue.

make next_entries() smarter about fitting Transactions into a Blob rob-solana/solana
2 participants
@mvines @rob-solana