Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Rename MEILI_MAX_MDB_SIZE env var and --max-mdb-size option flag #48

Merged
merged 6 commits into from
Jun 16, 2021

Conversation

gmourier
Copy link
Member

@gmourier gmourier commented Jun 14, 2021

I. Summary

This specification is written to rename the MEILI_MAX_MDB_SIZE environnement variable and --max-mdb-size option flag to MEILI_MAX_INDEX_SIZE and --max-index-size.

II. Motivation

Since version 0.21 no longer has a main LMDB database for all indexes but one LMBD database per index, the concept of a main database no longer exists internally.

@gmourier gmourier added the Ready For Review Feature specification must be reviewed. label Jun 14, 2021
text/0048-rename-max-mdb-size-var.md Outdated Show resolved Hide resolved
@gmourier gmourier changed the title Rename MAX_MDB_SIZE configuration var Rename MAX_MDB_SIZE env var and --max-mdb-size option flag Jun 16, 2021
@gmourier gmourier merged commit f80266c into main Jun 16, 2021
@gmourier gmourier deleted the rename-max-mdb-size-var branch June 16, 2021 16:00
@gmourier gmourier added Ready To Be Implemented Feature specification is ready to be implemented. and removed Ready For Review Feature specification must be reviewed. labels Jun 16, 2021
@gmourier gmourier changed the title Rename MAX_MDB_SIZE env var and --max-mdb-size option flag Rename MEILI_MAX_MDB_SIZE env var and --max-mdb-size option flag Jun 17, 2021
@gmourier gmourier added Implemented Feature specification has been implemented. and removed Ready To Be Implemented Feature specification is ready to be implemented. labels Jun 17, 2021
gmourier added a commit that referenced this pull request Aug 17, 2021
* Intialize specification

* Apply suggestions from code review

Co-authored-by: Clémentine Urquizar <clementine@meilisearch.com>

* update specification

* add 0.20 error description in specification file

* remove 0.21 current state missing http error

* update title and add date

Co-authored-by: Clémentine Urquizar <clementine@meilisearch.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Implemented Feature specification has been implemented. Q2:2021 v0.21
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants