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

Performance optimization of CurrentLineVerifier #21

Merged
merged 2 commits into from
Apr 12, 2021

Conversation

lechu445
Copy link
Contributor

@lechu445 lechu445 commented Apr 9, 2021

before:

| Method |     Mean |     Error |    StdDev | Gen 0 | Gen 1 | Gen 2 | Allocated |
|------- |---------:|----------:|----------:|------:|------:|------:|----------:|
|   Read | 6.077 ms | 0.0869 ms | 0.0813 ms |     - |     - |     - |   7.92 KB |

after:

| Method |     Mean |     Error |    StdDev | Gen 0 | Gen 1 | Gen 2 | Allocated |
|------- |---------:|----------:|----------:|------:|------:|------:|----------:|
|   Read | 5.722 ms | 0.0571 ms | 0.0534 ms |     - |     - |     - |   7.92 KB |

@lechu445 lechu445 force-pushed the lb_optimize_currentlineverifier branch 2 times, most recently from 4114a09 to 3c018c7 Compare April 12, 2021 06:18
before:
```
| Method |     Mean |     Error |    StdDev | Gen 0 | Gen 1 | Gen 2 | Allocated |
|------- |---------:|----------:|----------:|------:|------:|------:|----------:|
|   Read | 6.077 ms | 0.0869 ms | 0.0813 ms |     - |     - |     - |   7.92 KB |
```

after:
```
| Method |     Mean |     Error |    StdDev | Gen 0 | Gen 1 | Gen 2 | Allocated |
|------- |---------:|----------:|----------:|------:|------:|------:|----------:|
 |   Read | 5.722 ms | 0.0571 ms | 0.0534 ms |     - |     - |     - |   7.92 KB |
```
@lechu445 lechu445 force-pushed the lb_optimize_currentlineverifier branch from 3c018c7 to 3ffc79b Compare April 12, 2021 06:20
@wmaryszczak wmaryszczak merged commit 0c111ec into anixe:master Apr 12, 2021
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