Open Bug 1287162 Opened 8 years ago Updated 8 years ago

Consider switching FTL parser to Rust

Categories

(L20n :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

People

(Reporter: zbraniecki, Unassigned)

References

Details

We have a POC of the Rust parser for FTL at https://github.com/zbraniecki/l20n.rs and endoli is working on extending it in https://github.com/endoli/message-format.rs

First tests indicate that Rust parser is around 100 times faster. Once Gecko has reliable support for Rust we may want to switch to Rust parser.
Blocks: 1280674
Depends on: 1284816
gandalf, should this really block gecko-l20n? I'm leaning towards no, unless we find a hard blocker here.
Flags: needinfo?(gandalf)
No strong opinion here. I wanted to "consider", and that means that once we have things in place, measure Parser cost and if it's significant enough, and plugging in rust is easy enough, we can switch, if not, remove from the dependency or resolve as WONTFIX.

But if it messes with your dependency tree, feel free to remove it :) I'm sure we'll remember about this option anyway as we go through perf testing.
Flags: needinfo?(gandalf)
'k, let's move this out of the hot path for now.
No longer blocks: 1280674
You need to log in before you can comment on or make changes to this bug.