[PR #2387] [MERGED] Improve issue search #16380

Closed
opened 2025-11-02 12:09:17 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/go-gitea/gitea/pull/2387
Author: @ethantkoenig
Created: 8/25/2017
Status: Merged
Merged: 9/16/2017
Merged by: @lafriks

Base: masterHead: issue_indexer


📝 Commits (5)

  • b616585 Improve issue indexer
  • 8239ffc Fix new issue sqlite bug
  • 40dd00b Different test indexer paths for each db
  • ff0317d Add integration indexer paths to make clean
  • 503a0dc Merge branch 'master' into issue_indexer

📊 Changes

122 files changed (+15275 additions, -1453 deletions)

View changed files

📝 .gitignore (+3 -0)
📝 Makefile (+5 -1)
📝 integrations/integration_test.go (+8 -1)
📝 integrations/issue_test.go (+38 -5)
📝 integrations/mysql.ini.tmpl (+3 -0)
📝 integrations/pgsql.ini.tmpl (+3 -0)
📝 integrations/sqlite.ini (+3 -0)
📝 models/fixtures/issue.yml (+5 -5)
📝 models/issue.go (+17 -12)
📝 models/issue_comment.go (+20 -4)
📝 models/issue_indexer.go (+40 -124)
📝 models/pull.go (+2 -0)
📝 modules/indexer/indexer.go (+33 -4)
modules/indexer/issue.go (+143 -0)
📝 routers/init.go (+1 -2)
📝 routers/repo/issue.go (+2 -1)
📝 vendor/github.com/blevesearch/bleve/README.md (+25 -20)
vendor/github.com/blevesearch/bleve/analysis/analyzer/custom/custom.go (+145 -0)
vendor/github.com/blevesearch/bleve/analysis/analyzer/simple/simple.go (+0 -46)
vendor/github.com/blevesearch/bleve/analysis/token/unicodenorm/unicodenorm.go (+79 -0)

...and 80 more files

📄 Description

Make improvements to issue search: (see https://github.com/go-gitea/gitea/pull/530#pullrequestreview-23520746)

  • Add support for searching comment contents
  • Add unicode normalization and tokenization
  • Move indexer specific code out of models and into a separate module (where it can potentially be reused in the future)
  • Update bleve dependency
  • Add integration tests for issue search

Don't let the big add/removal figures scare you; that's because of the updated dependency


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/go-gitea/gitea/pull/2387 **Author:** [@ethantkoenig](https://github.com/ethantkoenig) **Created:** 8/25/2017 **Status:** ✅ Merged **Merged:** 9/16/2017 **Merged by:** [@lafriks](https://github.com/lafriks) **Base:** `master` ← **Head:** `issue_indexer` --- ### 📝 Commits (5) - [`b616585`](https://github.com/go-gitea/gitea/commit/b616585bb2c642e27941e48bbbc1e875b290a4ab) Improve issue indexer - [`8239ffc`](https://github.com/go-gitea/gitea/commit/8239ffca71076d40747b76dad714fca01247c3d5) Fix new issue sqlite bug - [`40dd00b`](https://github.com/go-gitea/gitea/commit/40dd00b7cc98741a00d1578923e5e6803aa351b8) Different test indexer paths for each db - [`ff0317d`](https://github.com/go-gitea/gitea/commit/ff0317d66815737ee3c27d1990e7564f8ef1194d) Add integration indexer paths to make clean - [`503a0dc`](https://github.com/go-gitea/gitea/commit/503a0dc0f4838256508ff55a0be23034dfee4730) Merge branch 'master' into issue_indexer ### 📊 Changes **122 files changed** (+15275 additions, -1453 deletions) <details> <summary>View changed files</summary> 📝 `.gitignore` (+3 -0) 📝 `Makefile` (+5 -1) 📝 `integrations/integration_test.go` (+8 -1) 📝 `integrations/issue_test.go` (+38 -5) 📝 `integrations/mysql.ini.tmpl` (+3 -0) 📝 `integrations/pgsql.ini.tmpl` (+3 -0) 📝 `integrations/sqlite.ini` (+3 -0) 📝 `models/fixtures/issue.yml` (+5 -5) 📝 `models/issue.go` (+17 -12) 📝 `models/issue_comment.go` (+20 -4) 📝 `models/issue_indexer.go` (+40 -124) 📝 `models/pull.go` (+2 -0) 📝 `modules/indexer/indexer.go` (+33 -4) ➕ `modules/indexer/issue.go` (+143 -0) 📝 `routers/init.go` (+1 -2) 📝 `routers/repo/issue.go` (+2 -1) 📝 `vendor/github.com/blevesearch/bleve/README.md` (+25 -20) ➕ `vendor/github.com/blevesearch/bleve/analysis/analyzer/custom/custom.go` (+145 -0) ➖ `vendor/github.com/blevesearch/bleve/analysis/analyzer/simple/simple.go` (+0 -46) ➕ `vendor/github.com/blevesearch/bleve/analysis/token/unicodenorm/unicodenorm.go` (+79 -0) _...and 80 more files_ </details> ### 📄 Description Make improvements to issue search: (see https://github.com/go-gitea/gitea/pull/530#pullrequestreview-23520746) - Add support for searching comment contents - Add unicode normalization and tokenization - Move indexer specific code out of models and into a separate module (where it can potentially be reused in the future) - Update bleve dependency - Add integration tests for issue search Don't let the big add/removal figures scare you; that's because of the updated dependency --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2025-11-02 12:09:17 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#16380