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

Add AI.INFO command #92

Closed
lantiga opened this issue Mar 19, 2019 · 4 comments
Closed

Add AI.INFO command #92

lantiga opened this issue Mar 19, 2019 · 4 comments

Comments

@lantiga
Copy link
Contributor

lantiga commented Mar 19, 2019

We should write the equivalent of the Redis INFO command, to make it easy to write plugin for Prometheus, Telegraf, etc for monitoring (time to compute, number of requests, memory, state of cuda allocations, etc)

@lantiga
Copy link
Contributor Author

lantiga commented Oct 30, 2019

Use Redis module INFO API

RedisModule_RegisterInfoFunc
RedisModule_InfoAddSection
RedisModule_InfoBeginDictField
RedisModule_InfoEndDictField
RedisModule_InfoAddFieldString
RedisModule_InfoAddFieldCString
RedisModule_InfoAddFieldDouble
RedisModule_InfoAddFieldLongLong
RedisModule_InfoAddFieldULongLong

@lantiga
Copy link
Contributor Author

lantiga commented Dec 6, 2019

Decision is still implement AI.INFO and replicate the same info using the INFO API when Redis 6 is out.

@lantiga
Copy link
Contributor Author

lantiga commented Dec 26, 2019

Addressing this in #259

@lantiga lantiga moved this from To do to In progress in RedisAI 1.0 GA Dec 26, 2019
@filipecosta90
Copy link
Collaborator

@lantiga closing this since it was already merged 👍

@filipecosta90 filipecosta90 moved this from In progress to Done in RedisAI 1.0 GA Feb 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

No branches or pull requests

4 participants