-
Notifications
You must be signed in to change notification settings - Fork 7
Add metrics and limiter small refactoring #159
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
check-spelling found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
This comment has been minimized.
This comment has been minimized.
|
The examples of metrics for a single query |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@check-spelling-bot Report🔴 Please reviewSee the 📂 files view, the 📜action log, or 📝 job summary for details.Unrecognized words (11)These words are not needed and should be removedpersisntTo accept these unrecognized words as correct and remove the previously acknowledged and now absent words, you could run the following commands... in a clone of the git@github.com:open-gpdb/yproxy.git repository curl -s -S -L 'https://raw.githubusercontent.com/check-spelling/check-spelling/main/apply.pl' |
perl - 'https://github.com/open-gpdb/yproxy/actions/runs/21069101591/attempts/1' &&
git commit -m 'Update check-spelling metadata'OR To have the bot accept them for you, comment in the PR quoting the following line: Available 📚 dictionaries could cover words (expected and unrecognized) not in the 📘 dictionaryThis includes both expected items (238) from .github/actions/spelling/expect.txt and unrecognized words (11)
Consider adding them (in with:
extra_dictionaries: |
cspell:golang/dict/go.txt
cspell:node/dict/node.txt
cspell:php/dict/php.txt
cspell:python/src/python/python-lib.txt
cspell:dotnet/dict/dotnet.txtTo stop checking additional dictionaries, add (in check_extra_dictionaries: ""Pattern suggestions ✂️ (9)You could add these patterns to Alternatively, if a pattern suggestion doesn't make sense for this project, add a Notices ℹ️ (1)See the 📂 files view, the 📜action log, or 📝 job summary for details.
See ℹ️ Event descriptions for more information. If the flagged items are 🤯 false positivesIf items relate to a ...
🚂 If you're seeing this message and your PR is from a branch that doesn't have check-spelling, |
Calculate the number of successful and errored requests and the number of processing items. Also measure latencies and sizes for requests:
It's not the whole list, but I think the essential methods. For measure latency, I delete total_time to data size, so measure processing time of 1 bite. Time measures in nanoseconds.
Default port for gather data is 2112, use
curl localhost:2112/metrics