Skip to content

Track remaining tasks to be "production ready" #5

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

Open
shakuzen opened this issue Nov 28, 2017 · 1 comment
Open

Track remaining tasks to be "production ready" #5

shakuzen opened this issue Nov 28, 2017 · 1 comment

Comments

@shakuzen
Copy link

We are currently considering using this plugin to integrate tracing from requests to Kong with the rest of our system. The readme has the following warning it in now.

This is proof of concept code, not production ready

What tasks/features are lacking currently for this to be production ready? If we could make that clear and track them here, people using this plugin and others in the community can more easily help make it production ready.

One thing I think I see by reading the code (though I'm not familiar with Kong or Lua at all) is it looks like it is sending the span data on each request instead of batching. I don't know how easy or possible it is, but batching sending spans could improve performance of reporting spans to Zipkin.

@sfhardman
Copy link
Owner

The warning is just there because I've personally only used Zipkin in proof-of-concept / prototype scenarios so far. I'm probably not going to get to a production deployment for several months.

I'm not aware of any issues preventing production use. Batching would be nice, but not sure how feasible it is (might require writing to the Postgres/Cassandra backend?). The only other nice-to-have I can think of is reporting the response code to Zipkin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants