Closed Larkooo closed 1 week ago
Ohayo, sensei! The changes in this pull request involve renaming fields in several structs to improve clarity, specifically changing entity
to ty
to better reflect their purpose as type descriptors. Additionally, the Service
struct in multiple files has been simplified by removing dependencies on database connections and model caches, streamlining the service's construction and functionality. The publish_updates
methods have also been refactored to rely on in-memory data rather than database queries, enhancing the overall efficiency of the code.
Files | Change Summary |
---|---|
crates/torii/core/src/query_queue.rs , crates/torii/core/src/sql.rs |
Renamed entity to ty in DeleteEntityQuery struct and related data structures, reflecting a shift towards a more structured representation of entity types. |
crates/torii/grpc/src/server/subscriptions/entity.rs , .../event_message.rs |
Simplified Service struct by removing Pool<Sqlite> and Arc<ModelCache> dependencies, and refactored publish_updates methods to utilize in-memory data. |
#2397: Modifications to the QueryQueue
struct and introduction of a new QueryType
enum, related to the renaming of the entity
field to ty
.
#2428: Introduction of a new DeleteEntityQuery
struct and a corresponding variant in the QueryType
enum, relevant to the changes in the DeleteEntityQuery
struct.
Attention: Patch coverage is 42.30769%
with 15 lines
in your changes missing coverage. Please review.
Project coverage is 68.36%. Comparing base (
82a23a5
) to head (d67dc75
). Report is 3 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Summary by CodeRabbit
New Features
Bug Fixes
Refactor