snapcraft does not use the C yaml parser

Bug #1787373 reported by John Lenton
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Snapcraft
Status tracked in Trunk
Legacy
Fix Committed
Medium
Kyle Fazzari
Trunk
Fix Released
Medium
Kyle Fazzari

Bug Description

PyYAML includes a C implementation of the yaml parser. However, yaml.safe_load does not use it: for that you need to use yaml.load(..., Loader=yaml.CSafeLoader). As snapcraft always does the former, it's very slow for big yamls.

Granted big yamls should be avoided, but the performance difference is big enough to notice even in not-so-huge ones.

Revision history for this message
Sergio Schvezov (sergiusens) wrote :
Changed in snapcraft:
milestone: none → 2.43.1
status: New → In Progress
assignee: nobody → Kyle Fazzari (kyrofa)
importance: Undecided → Medium
Changed in snapcraft:
milestone: 2.43.1 → 2.43.2
Changed in snapcraft:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.