meilisearch/MeiliSearch

Enable LTO again

Offen

#2.718 geöffnet am 29.08.2022

Auf GitHub ansehenContributor Guide
 (7 Kommentare) (1 Reaktion) (0 zugewiesene Personen)Rust (2.542 Forks)batch import
enhancementhelp wantedperformancetech discussion

Repository-Metriken

Stars
 (57.572 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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!

Contributor Guide

Research-Richtung
Recherchieren Sie, wie LTO in Rust nur für Nicht aarch64 Ziele bedingt aktiviert werden kann. Schauen Sie sich Cargo Build Skripte oder bedingte Kompilierung mit `cfg` Attributen an. Erwägen Sie die Verwendung von `#[cfg(target arch = "aarch64")]`, um LTO für diese Architektur zu deaktivieren.
Tech Stack
rust
Domain
backend
Issue Type
Performance
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
RustGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65