-
-
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
[WIP] Implement Elastic Search Code Indexer #7592
Conversation
Dependency: olivere/elastic, added to vendor and go mod
index: proposed change: code-indexer -> gitea mapping type: proposed change: indexer-data -> code
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.
It looks like I've been PR'ing on the same subject without realizing you were already working on it. For that I apologize. I leave you here a couple of highlights about my findings.
"type": custom.Name, | ||
"char_filters": []string{}, | ||
"tokenizer": unicode.Name, | ||
"token_filters": []string{unicodeNormalizeName, camelcase.Name, lowercase.Name, unique.Name}, |
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.
&IndexerData{ | ||
RepoID: repo.ID, | ||
Filepath: update.Filename, | ||
Content: string(fileContents), |
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.
Is this PR still alive? Otherwise I'd like to take #3670. |
Yes, planning on finishing it. (Haven't gotten a chance to work on it recently) |
@jeffliu27 do you had some time for this feature?. |
@jeffliu27 are you still working on this? |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 months. Thank you for your contributions. |
@lunny can this here be implemented with the same logic as you are implementing the issue indexer? |
@Cellebyte Yes. I will do that and replace this one if my previous issue indexer merged. |
@jeffliu27 Any news on your diff? It seems your code has conflicts. |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 months. Thank you for your contributions. |
This pull request has been automatically closed because of inactivity. You can re-open it if needed. |
Creates Issue indexer as a separate package: "codes" (will be changed to code later).
Added elastic search support for indexer instead of "bleve"
Ref issue: #6648