nginxinc / crossplane

Quick and reliable way to convert NGINX configurations into JSON and back.
Apache License 2.0
717 stars 86 forks source link

Implement Cython #3

Closed aluttik closed 6 years ago

aluttik commented 7 years ago

Consider adding Cython to this project for speedups. http://cython.readthedocs.io/en/latest/src/userguide/source_files_and_compilation.html

aluttik commented 6 years ago

So I tried this by compiling the source files but I saw no real speedups even on gigantic configs. I'm guessing it's probably because crossplane relies so heavily on generators. Perhaps if we were to rewrite most of the functions in C code we would get better results, but that'd require a huge overhaul of the current code and would probably make it much harder to read/contribute to. So for now I'm tabling this idea.