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

Track shard-level stats #2007

Merged
merged 2 commits into from
Mar 18, 2015
Merged

Track shard-level stats #2007

merged 2 commits into from
Mar 18, 2015

Conversation

otoolep
Copy link
Contributor

@otoolep otoolep commented Mar 18, 2015

No description provided.

@otoolep
Copy link
Contributor Author

otoolep commented Mar 18, 2015

$ curl -G http://localhost:8086/query?pretty=true --data-urlencode "db=internal" --data-urlencode "q=SHOW STATS"
{
    "results": [
        {
            "series": [
                {
                    "name": "server",
                    "columns": [
                        "shardsCreated",
                        "writeSeriesMessageTx",
                        "broadcastMessageTx",
                        "broadcastMessageRx",
                        "batchWriteRx",
                        "pointWriteRx"
                    ],
                    "values": [
                        [
                            1
                        ],
                        [
                            1
                        ],
                        [
                            4
                        ],
                        [
                            4
                        ],
                        [
                            1
                        ],
                        [
                            1
                        ]
                    ]
                },
                {
                    "name": "shard",
                    "columns": [
                        "writeSeriesMessageRx",
                        "shardBytes",
                        "shardWrite"
                    ],
                    "values": [
                        [
                            1
                        ],
                        [
                            26
                        ],
                        [
                            1
                        ]
                    ]
                }
            ]
        }
    ]
}

@otoolep
Copy link
Contributor Author

otoolep commented Mar 18, 2015

Looking at build failure.

@otoolep
Copy link
Contributor Author

otoolep commented Mar 18, 2015

Green build. Unrelated test failure, filed as ticket #2009

Fields: make(map[string]interface{}),
}
st.Walk(func(k string, v int64) {
point.Fields[k] = int(v)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just curious, why int(v)? isn't v already an int?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess int64 != int

@rothrock
Copy link
Contributor

Looks good to me.

otoolep added a commit that referenced this pull request Mar 18, 2015
@otoolep otoolep merged commit a35309c into master Mar 18, 2015
@otoolep otoolep deleted the more_stats branch March 18, 2015 23:07
@sanga
Copy link
Contributor

sanga commented Mar 19, 2015

isn't db=internal redundant here btw? Seems like an implementation detail that the user needn't care about.

@otoolep
Copy link
Contributor Author

otoolep commented Mar 19, 2015

Users need to know the database so they can query the stats via InfluxQL.

On Thursday, March 19, 2015, Tim Sampson notifications@github.com wrote:

isn't db=internal redundant here btw? Seems like an implementation detail
that the user needn't care about.


Reply to this email directly or view it on GitHub
#2007 (comment).

mark-rushakoff pushed a commit that referenced this pull request Jan 11, 2019
Match shape of get/authorizations to backend response
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

Successfully merging this pull request may close these issues.

3 participants