test1234
/
gitea
forked from gitea/gitea
1
0
Fork 0
Code Pull Requests Activity

fix: Elasticsearch: Request Entity Too Large #28117 (#29062) 

Fix for gitea putting everything into one request without batching and
sending it to Elasticsearch for indexing as issued in #28117

This issue occured in large repositories while Gitea tries to 
index the code using ElasticSearch.

I've applied necessary changes that takes batch length from below config
(app.ini)
```
[queue.code_indexer]
BATCH_LENGTH=<length_int>
```
and batches all requests to Elasticsearch in chunks as configured in the
above config
This commit is contained in:
dark-angel 2024-02-07 14:27:16 +05:30 committed by GitHub
parent 2bac85dc33
commit 5c0fc90872
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
modules/indexer/code/elasticsearch/elasticsearch.go
View File

@ -180,12 +180,18 @@ func (b *Indexer) Index(ctx context.Context, repo *repo_model.Repository, sha st
} }
if len(reqs) > 0 { if len(reqs) > 0 {
esBatchSize := 50
for i := 0; i < len(reqs); i += esBatchSize {
_, err := b.inner.Client.Bulk(). _, err := b.inner.Client.Bulk().
Index(b.inner.VersionedIndexName()). Index(b.inner.VersionedIndexName()).
Add(reqs...). Add(reqs[i:min(i+esBatchSize, len(reqs))]...).
Do(ctx) Do(ctx)
if err != nil {
return err return err
} }
}
}
return nil return nil
} }