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

Need additional attributes in NodeInfo of node.go file and QueueInfo of queue.go #115

Closed
Durgababu opened this issue Jun 8, 2018 · 4 comments
Milestone

Comments

@Durgababu
Copy link
Contributor

Durgababu commented Jun 8, 2018

Hi,
I need few more attributes in NodeInfo and QueueInfo structures, which are available in same json responses.

In NodeInfo structure of node.go file , I need below two attributes additionally ...

ProcUsed int json:"proc_used"
ProcTotal int json:"proc_total"

In QueueInfo structures of queue.go file, need below attribute additionally....

ActiveConsumers int64 json:"active_consumers"

Only these lines should be included in the code. No further changes required as we have these attributes available in response jsons.

Please do the changes and commit, so that i can use the latest package.

@jordanbcooper
Copy link
Contributor

jordanbcooper commented Jun 8, 2018

I actually opened a PR to implement proc_used and proc_total, haven't heard back yet about merging.

@jordanbcooper
Copy link
Contributor

@Durgababu -- attributes in my above comment were merged.

@Durgababu
Copy link
Contributor Author

@jordanbcooper -- Yeah, i checked. Your changes are merged. But sill i need one more change to be merged. Let me create new merge request.

Below are my changes...
In QueueInfo structures of queue.go file, need below attribute additionally....

ActiveConsumers int64 json:"active_consumers"

@michaelklishin
Copy link
Owner

Addressed in #113 and #117.

@michaelklishin michaelklishin added this to the 1.5.0 milestone Feb 9, 2019
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

3 participants