Skip to content

ReBAC: Optimise when creating relations#98

Merged
uatuko merged 18 commits intomainfrom
feature/rebac-optimise
May 14, 2024
Merged

ReBAC: Optimise when creating relations#98
uatuko merged 18 commits intomainfrom
feature/rebac-optimise

Conversation

@uatuko
Copy link
Copy Markdown
Owner

@uatuko uatuko commented May 9, 2024

This change utilises the optimise flag to compute and store derived relations when creating a new relation. The cost_limit can be used to avoid expensive DB operations during an insert.

💡 If the cost of storing derived tuples is larger than the cost_limit, the response will have a negative cost indicating the process has been aborted.

@codecov
Copy link
Copy Markdown

codecov bot commented May 9, 2024

Codecov Report

Attention: Patch coverage is 90.54054% with 7 lines in your changes are missing coverage. Please review.

Project coverage is 92.90%. Comparing base (703fddf) to head (a080023).

Files Patch % Lines
src/svc/relations.cpp 87.75% 2 Missing and 4 partials ⚠️
src/db/tuples.cpp 95.65% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #98      +/-   ##
==========================================
+ Coverage   92.79%   92.90%   +0.11%     
==========================================
  Files          18       18              
  Lines        1069     1128      +59     
  Branches      115      126      +11     
==========================================
+ Hits          992     1048      +56     
  Misses         61       61              
- Partials       16       19       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

neculalaura
neculalaura previously approved these changes May 9, 2024
@uatuko uatuko marked this pull request as ready for review May 14, 2024 08:49
@uatuko uatuko merged commit c7a50d3 into main May 14, 2024
@uatuko uatuko deleted the feature/rebac-optimise branch May 14, 2024 12:40
@uatuko uatuko mentioned this pull request May 14, 2024
14 tasks
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.

2 participants