Skip to content

perf: Added benchmark test and limited expression compiling to once unless required#80

Merged
leeqvip merged 2 commits intocasbin:masterfrom
rushitote:master
Jul 7, 2021
Merged

perf: Added benchmark test and limited expression compiling to once unless required#80
leeqvip merged 2 commits intocasbin:masterfrom
rushitote:master

Conversation

@rushitote
Copy link
Member

This is the earlier benchmark (LuaJIT) from GitHub Actions:

Raw:		Total ops = 10000		Time per op = 0.0005 ms
Basic Model:		Total ops = 10000		Time per op = 0.1157 ms
RBAC Model:		Total ops = 10000		Time per op = 0.2935 ms
RBAC Model Small:		Total ops = 1000		Time per op = 6.3911 ms
RBAC Model Medium:		Total ops = 100		Time per op = 63.5531 ms
RBAC Model Large:		Total ops = 10		Time per op = 670.6526 ms
RBAC Model with Resources:		Total ops = 1000		Time per op = 0.3454 ms
RBAC Model with Domains:		Total ops = 1000		Time per op = 0.4186 ms
ABAC Model:		Total ops = 1000		Time per op = 0.0311 ms
KeyMatch Model:		Total ops = 1000		Time per op = 0.4127 ms
RBAC Model with Deny:		Total ops = 1000		Time per op = 0.3503 ms
Priority Model:		Total ops = 1000		Time per op = 0.4927 ms

This is the current one:

Raw:		Total ops = 10000		Time per op = 0.0005 ms
Basic Model:		Total ops = 10000		Time per op = 0.0786 ms
RBAC Model:		Total ops = 10000		Time per op = 0.1393 ms
RBAC Model Small:		Total ops = 1000		Time per op = 1.1131 ms
RBAC Model Medium:		Total ops = 100		Time per op = 11.4998 ms
RBAC Model Large:		Total ops = 10		Time per op = 127.0854 ms
RBAC Model with Resources:		Total ops = 1000		Time per op = 0.2428 ms
RBAC Model with Domains:		Total ops = 1000		Time per op = 0.2113 ms
ABAC Model:		Total ops = 1000		Time per op = 0.0484 ms
KeyMatch Model:		Total ops = 1000		Time per op = 0.1511 ms
RBAC Model with Deny:		Total ops = 1000		Time per op = 0.3070 ms
Priority Model:		Total ops = 1000		Time per op = 0.2016 ms

rushitote added 2 commits July 7, 2021 12:41
Signed-off-by: Rushikesh Tote <rushi.tote@gmail.com>
Signed-off-by: Rushikesh Tote <rushi.tote@gmail.com>
@leeqvip
Copy link
Member

leeqvip commented Jul 7, 2021

Good job!

@leeqvip leeqvip merged commit 42fbfd4 into casbin:master Jul 7, 2021
@github-actions
Copy link

github-actions bot commented Jul 7, 2021

🎉 This PR is included in version 1.16.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@hsluoyz
Copy link
Member

hsluoyz commented Jul 7, 2021

@rushitote plz also consider adding Lua result to: https://casbin.org/docs/en/benchmark , we want to add a multi-language tab like code snipet then add benchmarks for different languages.

@rushitote
Copy link
Member Author

@hsluoyz Sure, I will do that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants