Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve speed of yaml emitter #30

Open
kostja opened this issue Sep 17, 2013 · 1 comment
Open

Improve speed of yaml emitter #30

kostja opened this issue Sep 17, 2013 · 1 comment
Labels
feature A new functionality good first issue Good for newcomers lua performance
Milestone

Comments

@kostja
Copy link
Contributor

kostja commented Sep 17, 2013

It takes over 1 second on my box to output a large Lua table (100k records) to the administrative console. The bottleneck is most likely in the new yaml emitter. In particular, the emitter allocates memory for every node with malloc, so it performs 100k malloc calls.

How it could possibly be improved:

  • replace malloc with palloc
  • look at other bottlenecks and see what can be improved in yaml token analysis
@ilnarb
Copy link

ilnarb commented Sep 18, 2013

  • just use modern allocator, like tcmalloc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A new functionality good first issue Good for newcomers lua performance
Projects
None yet
Development

No branches or pull requests

3 participants