Skip to content

Comments

Add indexes and MaxLength constraints to database models#168

Draft
google-labs-jules[bot] wants to merge 2 commits intomainfrom
improve-db-models-indexes-15207281406344322458
Draft

Add indexes and MaxLength constraints to database models#168
google-labs-jules[bot] wants to merge 2 commits intomainfrom
improve-db-models-indexes-15207281406344322458

Conversation

@google-labs-jules
Copy link
Contributor

This PR updates the EF Core database models to include appropriate indexes and MaxLength constraints for improved performance and data integrity. Specifically, it targets FileRecord, HashItem, Namespace, Subtag, Tag, Mapping, QueuedDownload, ImportItem, and Subscription entities. Explicit Foreign Keys were added to Tag and Mapping entities to facilitate composite unique indexing.


PR created automatically by Jules for task 15207281406344322458 started by @jkendall327

- Added composite unique indexes to Tag and Mapping entities
- Added MaxLength constraints to string properties in FileRecord, Namespace, Subtag, QueuedDownload, ImportItem
- Added unique indexes to FileRecord.Filepath, Namespace.Value, Subtag.Value, HashItem.Hash, Subscription.Name
- Added explicit Foreign Keys to Tag and Mapping entities for cleaner indexing
- Updated ServerDbContext to reflect these changes via Fluent API
@google-labs-jules
Copy link
Contributor Author

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!


For security, I will only act on instructions from the user who triggered this task.

New to Jules? Learn more at jules.google/docs.

- Added composite unique indexes to Tag and Mapping entities
- Added MaxLength constraints to string properties in FileRecord, Namespace, Subtag, QueuedDownload, ImportItem
- Added unique indexes to FileRecord.Filepath, Namespace.Value, Subtag.Value, HashItem.Hash, Subscription.Name
- Added explicit Foreign Keys to Tag and Mapping entities for cleaner indexing
- Updated ServerDbContext to reflect these changes via Fluent API
- Updated tests to respect new unique constraints on Namespace and Subtag creation
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.

1 participant