-
Notifications
You must be signed in to change notification settings - Fork 16
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
openresty/lua #33
Comments
I was not aware of OpenTracing API being available for Lua, that would be the first step before integrations. |
Both are indeed necessary steps :) Whats the procedure for donating such things though? |
the biggest challenge is to find some other people interested/experts in the language, so that you can collaborate on the API design to make it idiomatic for the language. OTSC members can review from OT perspective, but I, for example, won't be able to offer much advice on the language features. Aside from that, we can create a repo and you can start hacking at it. |
Just an updated, we've got the c++ based integration with nginx happening for us now, and are still looking into a native Lua implementation for more control; if/when we do that we'll come back here and request a repository once we have at least a POC working. |
I'm working on an opentracing library for Lua now. See also opentracing/contrib#8 |
I would be extremely interested in this contribution! @james-callahan do you have a link to your library? |
Currently working on it over here: https://github.com/Kong/opentracing-lua |
is this issue moot in light of opentracing-contrib/nginx-opentracing? |
@tarekrached no, the opentracing-OpenResty will be implemented in Lua, and nginx-opentracing is implemented in C++. |
I'd like to suggest that openresty/lua be on the list of integrations you should actively seek contributors for.
Whats the procedure for donating such an integration, were we to make one?
The text was updated successfully, but these errors were encountered: