-
-
Notifications
You must be signed in to change notification settings - Fork 199
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
sjsonnet integration #794
Comments
Certainly something worth discussing! There's quite a few implementations out there. While I don't think it's worth supporting them at this stage, perhaps we can think of a generic way of running external commands that's a bit more sophisticated than the |
There really are quite a few implementations out there... I had no idea. For example, the rust impl jrsonnet has even more impressive (and more robust!) benchmarking results, and it even has python bindings! |
Looks like Some quick benchmarking results against the kapitan examples:
So.. barely faster than gojsonnet, and also incomplete (or incompatible?) |
Can you share the exception information? We can report the issue to jrsonnet project. |
Unfortunately I no longer have access to the inventory anymore (moved to a different company) so I’m unable to replicate. |
This issue is stale because it has been open for 1 year with no activity. |
This issue is raised purely for discovery and discussion purposes.
Databricks have been building a jsonnet implementation that runs on the JVM: sjsonnet. Interestingly, their benchmark advertises a ~200x performance boost over jsonnet, and a ~40x performance boost over go-jsonnet.
Unfortunately, there are no python bindings for sjsonnet and so it wouldn't just be a drop-in replacement into kapitan. Raising this just to highlight the fact that such a thing exists.
A tiny bit of solutionising: a search for python + jvm yields the jvm python module. The docs don't show any examples of how it should/could be used, and there's the fact that it's still beta and does not have a (visible) userbase, so I'm not sure if it even could be a good option... but it is a relatively new project.
The text was updated successfully, but these errors were encountered: