meilisearch/MeiliSearch

Enable LTO again

Aperta

#2718 aperta il 29 ago 2022

Vedi su GitHubGuida contributor
 (7 commenti) (1 reazione) (0 assegnatari)Rust (2542 fork)batch import
enhancementhelp wantedperformancetech discussion

Metriche repository

Star
 (57.572 stelle)
Metriche merge PR
 (Merge medio 11g 20h) (41 PR mergiate in 30 g)

Descrizione

We first enabled LTO in this PR https://github.com/meilisearch/meilisearch/pull/2690 to improve the performance of the search engine.

Then, we had to disable LTO in this PR https://github.com/meilisearch/meilisearch/pull/2717 because we were not able to compile the aarch64 binary anymore. See: https://github.com/meilisearch/meilisearch/actions/runs/2948932141

We would introduce LTO again and prevent using it when compiling the aarch64 binary. Some beginning of solution here

If anyone has a better solution, we would happy to read you!

Guida contributor

Direzione di ricerca
Ricerca come abilitare condizionalmente LTO solo per i target non aarch64 in Rust. Guarda gli script di build di Cargo o la compilazione condizionale usando gli attributi `cfg`. Considera l'uso di `#[cfg(target arch = "aarch64")]` per disabilitare LTO per quell'architettura.
Tech stack
rust
Dominio
backend
Tipo issue
Performance
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
RustGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65