-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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 more tests and docs for issue indexer, add db indexer type for searching from database #6144
Conversation
…arching from database
Codecov Report
@@ Coverage Diff @@
## master #6144 +/- ##
==========================================
- Coverage 38.85% 38.85% -0.01%
==========================================
Files 352 354 +2
Lines 50049 50112 +63
==========================================
+ Hits 19449 19472 +23
- Misses 27784 27822 +38
- Partials 2816 2818 +2
Continue to review full report at Codecov.
|
Will fix #5798 |
- `REPO_INDEXER_ENABLED`: **false**: 是否启用代码搜索(启用后会占用比较大的磁盘空间)。 | ||
- `REPO_INDEXER_PATH`: **indexers/repos.bleve**: 用于代码搜索的索引文件路径。 | ||
- `UPDATE_BUFFER_LEN`: **20**: 代码索引请求的缓冲区长度。 | ||
- `MAX_FILE_SIZE`: **1048576**: 进行解析的源代码文件的最大长度,小于才索引。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggest: 小于该值时才会索引, 或将“小于才索引去掉”。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done.
This PR add more tests and docs for issue indexer and added a new indexer type
db
which will only search database tableissue
andcomment
viaSQL
like. It's not effective but for a small deployment is enough.