Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add index for UpdatedEntity field of ImportNotification #137

Merged
merged 2 commits into from
Feb 13, 2025

Conversation

tjpeel-ee
Copy link
Contributor

The field is being used in PHA API queries. There are additional fields we are also filtering on so not sure if a composite index across all fields would be preferable?

For now, thought it made sense to at least include an index on UpdatedEntity.

t11omas
t11omas previously approved these changes Feb 13, 2025
t11omas
t11omas previously approved these changes Feb 13, 2025
Copy link

Code Coverage

Package Line Rate Branch Rate Health
Btms.Business 82% 73%
Btms.SyncJob 63% 45%
Btms.Metrics 82% 0%
Btms.Analytics 74% 73%
Btms.Backend.Data 65% 60%
Btms.Types.Ipaffs 95% 62%
TestDataGenerator 83% 80%
Btms.Backend 62% 39%
Btms.BlobService 38% 27%
Btms.Emf 11% 0%
Btms.Common 72% 56%
Btms.Azure 43% 100%
Btms.Consumers 81% 72%
Btms.Types.Alvs 85% 67%
Btms.Types.Gvms.Mapping 38% 25%
Btms.Types.Alvs.Mapping 81% 0%
Btms.Types.Ipaffs.Mapping 77% 50%
Btms.Types.Gvms 61% 100%
Btms.SensitiveData 82% 89%
Btms.Types.Alvs.Mapping.V1.Tests 0% 0%
Btms.Model 93% 94%
TestGenerator.IntegrationTesting.Backend 83% 71%
Summary 78% (10111 / 12972) 62% (1024 / 1646)

@tjpeel-ee tjpeel-ee merged commit 9a8fc4d into main Feb 13, 2025
3 checks passed
@tjpeel-ee tjpeel-ee deleted the PHAIN-106-updatedEntity-index branch February 13, 2025 15:19
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