opensearch-project / neural-search

Plugin that adds dense neural retrieval into the OpenSearch ecosytem
Apache License 2.0
57 stars 58 forks source link

Refactor integ tests for aggregations #699

Closed martin-gaievski closed 2 months ago

martin-gaievski commented 2 months ago

We need to refactor current integ tests that were created for aggregations (https://github.com/opensearch-project/neural-search/pull/632). Currently there are 20+ aggregations that are execute with and without concurrent segment search each. They are split into 3 classes based on type of aggregations that are there.

At least we need to optimize tests for cases when concurrent segment search, so we don't need to flip that setting for every test. We can split each class into 2 classes, or we can create two separate indexes one with concurrent search and one without.