Skip to content

Create in-memory MDBX inside dirs.Tmp#5702

Merged
AlexeyAkhunov merged 3 commits into
develfrom
memdb_dir
Oct 11, 2022
Merged

Create in-memory MDBX inside dirs.Tmp#5702
AlexeyAkhunov merged 3 commits into
develfrom
memdb_dir

Conversation

@yperbasis

@yperbasis yperbasis commented Oct 11, 2022

Copy link
Copy Markdown
Member

Previously "in-memory" MDBX instances for fork validation and mining were created inside os.TempDir(). We should create them inside Erigon's datadir so that the file permissions and the disk are the same as for the main database.

Prerequisite: ledgerwatch/erigon-lib#676.

yperbasis added a commit to ledgerwatch/erigon-lib that referenced this pull request Oct 11, 2022
@yperbasis yperbasis marked this pull request as ready for review October 11, 2022 15:00
@AlexeyAkhunov AlexeyAkhunov merged commit 7286a0f into devel Oct 11, 2022
@AlexeyAkhunov AlexeyAkhunov deleted the memdb_dir branch October 11, 2022 15:49
@AskAlexSharov

Copy link
Copy Markdown
Collaborator

@yperbasis you renamed etl-temp to temp, but forgot change const TmpDirName = "etl-temp"?

@AskAlexSharov

Copy link
Copy Markdown
Collaborator

probably need remove const TmpDirName because it's almost unused

yperbasis added a commit that referenced this pull request Oct 12, 2022
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.

3 participants