Skip to content

Bring crdt backup, config, and network config logic from 3.5 branch.#308

Open
itsafuu wants to merge 16 commits into
developfrom
dev_35todevelop
Open

Bring crdt backup, config, and network config logic from 3.5 branch.#308
itsafuu wants to merge 16 commits into
developfrom
dev_35todevelop

Conversation

@itsafuu

@itsafuu itsafuu commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@itsafuu itsafuu requested a review from henriqueaklein June 1, 2026 19:57

@henriqueaklein henriqueaklein left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

io_threads_ is probably the only error. The rest are suggestions/improvements.

Comment thread src/crdt/globaldb/globaldb.cpp Outdated
Comment thread src/crdt/globaldb/globaldb.cpp
Comment thread src/crdt/globaldb/globaldb.hpp Outdated
Comment thread src/crdt/globaldb/globaldb.hpp Outdated
Comment thread src/account/GeniusNode.hpp Outdated
Comment thread src/account/GeniusNode.cpp
Comment thread src/account/GeniusNode.hpp
Comment thread src/account/GeniusNode.cpp Outdated

@henriqueaklein henriqueaklein left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Check the duplicated code (probably result of merge errors) and more importantly, the crdt_datastore destructor needs the code that detects if the destructor was called by a worker thread, otherwise it will get stuck

Comment thread src/account/GeniusNode.cpp Outdated
Comment thread src/account/GeniusNode.cpp Outdated
Comment thread src/account/GeniusNode.cpp Outdated
Comment thread src/account/GeniusNode.cpp
Comment thread src/account/GeniusNode.cpp
Comment thread src/account/GeniusNode.cpp Outdated
Comment thread src/account/GeniusNode.cpp Outdated
Comment thread src/account/GeniusNode.cpp Outdated
Comment thread src/account/GeniusNode.cpp Outdated
Comment thread src/crdt/impl/crdt_datastore.cpp Outdated
Super-Genius added a commit that referenced this pull request Jun 9, 2026
- 8 new auto-filed test-failure bugs (#300-#307)
- 3 open PRs: #309 bridge_phase5, #308 crdt-3.5-backport, #299 memory-secure-storage
- Updated bridge, consensus perf, and stale-item sections
- Gate violations noted: no PR/issue templates exist in repo

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@EduMenges EduMenges force-pushed the develop branch 3 times, most recently from 128165d to afb4967 Compare June 10, 2026 20:13
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