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.

Sign up for GitHub

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

Jump to bottom

feat: trgm index for search #1245

Open
wants to merge 20 commits into
base: develop
Choose a base branch
from
Open

feat: trgm index for search #1245

wants to merge 20 commits into from
+1,915 −738

Conversation

tea-artist
Copy link
Contributor

No description provided.

@tea-artist tea-artist force-pushed the perf/search-trgm-index branch from a1a666d to 241caa3 Compare January 13, 2025 19:22
@tea-artist tea-artist added the preview This issue is already ready for preview label Jan 13, 2025
@github-actions GitHub Actions
Copy link

Deployment Status: ✅ Success
🔗 Preview URL: https://pr-1245.sealoshzh.site

@tea-artist tea-artist requested a review from caoxing9 January 14, 2025 02:53
@github-actions GitHub Actions
Copy link

Deployment Status: ✅ Success
🔗 Preview URL: https://pr-1245.sealoshzh.site

1 similar comment
@github-actions GitHub Actions
Copy link

Deployment Status: ✅ Success
🔗 Preview URL: https://pr-1245.sealoshzh.site

@caoxing9 caoxing9 force-pushed the perf/search-trgm-index branch 2 times, most recently from 059260d to fb2bdc7 Compare January 16, 2025 09:55
@caoxing9 caoxing9 force-pushed the perf/search-trgm-index branch from 2f55c38 to 4c1fda1 Compare January 16, 2025 11:18
@github-actions GitHub Actions
Copy link

Deployment Status: ✅ Success
🔗 Preview URL: https://pr-1245.sealoshzh.site

@caoxing9 caoxing9 force-pushed the perf/search-trgm-index branch 2 times, most recently from 908c3b4 to 06528e0 Compare January 16, 2025 13:37
@github-actions GitHub Actions
Copy link

Deployment Status: ✅ Success
🔗 Preview URL: https://pr-1245.sealoshzh.site

1 similar comment
@github-actions GitHub Actions
Copy link

Deployment Status: ✅ Success
🔗 Preview URL: https://pr-1245.sealoshzh.site

@caoxing9 caoxing9 force-pushed the perf/search-trgm-index branch from 06528e0 to 8d24aa5 Compare January 16, 2025 13:51
@github-actions GitHub Actions
Copy link

Deployment Status: ✅ Success
🔗 Preview URL: https://pr-1245.sealoshzh.site

@caoxing9 caoxing9 force-pushed the perf/search-trgm-index branch from 8d24aa5 to 088dae2 Compare January 16, 2025 14:43
@github-actions GitHub Actions
Copy link

Deployment Status: ✅ Success
🔗 Preview URL: https://pr-1245.sealoshzh.site

@github-actions GitHub Actions
Copy link

Deployment Status: ✅ Success
🔗 Preview URL: https://pr-1245.sealoshzh.site

@github-actions GitHub Actions
Copy link

Deployment Status: ✅ Success
🔗 Preview URL: https://pr-1245.sealoshzh.site

@github-actions GitHub Actions
Copy link

Deployment Status: ✅ Success
🔗 Preview URL: https://pr-1245.sealoshzh.site

@github-actions GitHub Actions
Copy link

Deployment Status: ✅ Success
🔗 Preview URL: https://pr-1245.sealoshzh.site

@coveralls
Copy link
Collaborator

coveralls commented Jan 17, 2025 •
edited
Loading

Pull Request Test Coverage Report for Build 12840858820

Details

  • 873 of 1012 (86.26%) changed or added relevant lines in 21 files are covered.
  • 15 unchanged lines in 3 files lost coverage.
  • Overall coverage increased (+0.2%) to 81.709%
Changes Missing Coverage Covered Lines Changed/Added Lines %
apps/nestjs-backend/src/db-provider/search-query/types.ts 0 1 0.0%
apps/nestjs-backend/src/db-provider/search-query/search-index-builder.postgres.ts 165 168 98.21%
apps/nestjs-backend/src/features/record/record.service.ts 29 35 82.86%
apps/nestjs-backend/src/db-provider/search-query/abstract.ts 16 26 61.54%
apps/nestjs-backend/src/db-provider/search-query/search-query.sqlite.ts 139 149 93.29%
apps/nestjs-backend/src/db-provider/search-query/search-query.postgres.ts 163 176 92.61%
apps/nestjs-backend/src/features/table/table-index.service.ts 190 204 93.14%
apps/nestjs-backend/src/features/aggregation/aggregation.service.ts 45 72 62.5%
apps/nestjs-backend/src/db-provider/search-query/search-index-builder.sqlite.ts 25 80 31.25%
Files with Coverage Reduction New Missed Lines %
apps/nestjs-backend/src/db-provider/search-query/search-query.postgres.ts 1 91.14%
apps/nestjs-backend/src/db-provider/search-query/abstract.ts 1 49.45%
apps/nestjs-backend/src/features/auth/local-auth/local-auth.service.ts 13 63.7%
Totals Coverage Status
Change from base Build 12830575244: 0.2%
Covered Lines: 31168
Relevant Lines: 38145
💛 - Coveralls

@github-actions GitHub Actions
Copy link

Deployment Status: ✅ Success
🔗 Preview URL: https://pr-1245.sealoshzh.site

@caoxing9 caoxing9 force-pushed the perf/search-trgm-index branch from dfc1fbe to c5e26c1 Compare January 17, 2025 13:59
@github-actions GitHub Actions
Copy link

Deployment Status: ✅ Success
🔗 Preview URL: https://pr-1245.sealoshzh.site

@caoxing9 caoxing9 force-pushed the perf/search-trgm-index branch from c5e26c1 to 7a12aff Compare January 17, 2025 14:44
@github-actions GitHub Actions
Copy link

Deployment Status: ✅ Success
🔗 Preview URL: https://pr-1245.sealoshzh.site

@caoxing9 caoxing9 force-pushed the perf/search-trgm-index branch from 7a12aff to 77d00e1 Compare January 18, 2025 01:58
@github-actions GitHub Actions
Copy link

Deployment Status: ✅ Success
🔗 Preview URL: https://pr-1245.sealoshzh.site

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@caoxing9 caoxing9 Awaiting requested review from caoxing9

Assignees
No one assigned
Labels
preview This issue is already ready for preview
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tea-artist @coveralls @caoxing9