Commit 6e95fc3
fix StreamStringTermsAggregatorTests by replacing RandomIndexWriter with plain IndexWriter opensearch-project#18956 WIP (opensearch-project#19404)
* fix opensearch-project#18956 WIP
- I replaced RandomIndexWriter to plain IndexWriter to avoid unexpected segments in test which are unable to handle more than single segment
- for other test I've added explicit segmenting. TODO It needs to be reviewed.
- Perhaps StreamingTermsAggregator should reject second segment via assert or there should be a proper publish/reset flow. TODO, WIP
so far this commit fixes the known reason for flakiness (I believe)
Signed-off-by: Mikhail Khludnev <[email protected]>
* newIndexWriterConfig() causes segmenting via flushes by bufferedDocs
Signed-off-by: Mikhail Khludnev <[email protected]>
* subAggs test goes single segment
Signed-off-by: Mikhail Khludnev <[email protected]>
* asserting single segment
Signed-off-by: Mikhail Khludnev <[email protected]>
---------
Signed-off-by: Mikhail Khludnev <[email protected]>1 parent ff62aeb commit 6e95fc3
File tree
1 file changed
+59
-31
lines changed- server/src/test/java/org/opensearch/search/aggregations/bucket/terms
1 file changed
+59
-31
lines changed
0 commit comments