server: include time units in app protos comments#92602
server: include time units in app protos comments#92602craig[bot] merged 1 commit intocockroachdb:masterfrom
Conversation
|
@lnhsingh We want to document these units in https://www.cockroachlabs.com/docs/stable/crdb-internal.html. |
Closes cockroachdb#89976 This commit includes the unit of time in the comments of fields in app_stats protos where the unit of time was not specified. Release note: None
b35e345 to
297b669
Compare
|
Created docs issue: https://cockroachlabs.atlassian.net/browse/DOC-6259 |
|
Build succeeded: |
|
Encountered an error creating backports. Some common things that can go wrong:
You might need to create your backport manually using the backport tool. error creating merge commit from 297b669 to blathers/backport-release-22.2-92602: POST https://api.github.com/repos/cockroachdb/cockroach/merges: 409 Merge conflict [] you may need to manually resolve merge conflicts with the backport tool. Backport to branch 22.2.x failed. See errors above. 🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is otan. |
Closes #89976
This commit includes the unit of time in the comments of fields in app_stats protos where the unit of time was not specified.
Release note: None