You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2021/12/05 12:19:25 UTC
[GitHub] [accumulo] EdColeman commented on pull request #2374: Cache the credentials because they are expensive to fetch
EdColeman commented on pull request #2374:
URL: https://github.com/apache/accumulo/pull/2374#issuecomment-986220077
Running an a large instance with 16 cores / 32 threads, 128G memory and using SSDs I see some improvement, but not as drastic as reported by others.
There seems to be two modes that occur with some runs being ~90 seconds faster on my instance. In 10 runs on the current main branch, the faster runs occurred twice with an average run time of 162.7 seconds, and eight runs with an average of 252.8 seconds.
Using this PR and 20 runs, the faster runs occurred five times with an average of 83.3 seconds (5.7 seconds faster). The slower runs occurred 15 times with an average of 240.3 seconds (12.5 seconds faster)
# Run time details
<details>
<summary>Click to expand!</summary>
## Run details main branch vs PR with cached credentials
| run | main | cached |
|-----|---------|-----|
| 1 | 256.651 | 242.218 |
| 2 | 164.723 | 155.704 |
| 3 | 250.717 | 241.091 |
| 4 | 250.772 | 239.305 |
| 5 | 160.739 | 245.015 |
| 6 | 253.339 | 155.78 |
| 7 | 253.828 | 238.795 |
| 8 | 254.756 | 241.777 |
| 9 | 250.276 | 236.324 |
| 10 | 251.828 | 160.737 |
| 11 | | 241.228 |
| 12 | | 238.235 |
| 13 | | 160.254 |
| 14 | | 239.816 |
| 15 | | 242.269 |
| 16 | | 152.27 |
| 17 | | 239.339 |
| 18 | | 240.57 |
| 19 | | 239.227 |
| 20 | | 239.218 |
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org