feat: attribute import, temporal properties, SET fix#379
Conversation
- Add import_attrs() to AttributeStore for bulk importing entity attributes - Add import_node_attrs()/import_relationship_attrs() to Graph - Allow temporal types (Datetime, Date, Time, Duration) in is_valid_property - Fix SET on relationships that exist in pending state
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 40 minutes and 32 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #379 +/- ##
==========================================
- Coverage 83.16% 83.02% -0.14%
==========================================
Files 109 110 +1
Lines 29327 29400 +73
==========================================
+ Hits 24389 24410 +21
- Misses 4938 4990 +52
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Summary
import_attrs()toAttributeStorefor bulk importing entity attributes (used by RDB decoder)import_node_attrs()/import_relationship_attrs()toGraphDatetime,Date,Time,Duration) inis_valid_propertyvalidationSETon relationships that exist in pending stateContext
Part of splitting #359 (RDB serialization) into smaller reviewable PRs. This PR extracts attribute import helpers and two bug fixes that are independent of the larger serialization changes.
Test plan
cargo test -p graphpassespytest tests/test_e2e.py -vv)