Sandesh Kumar

Results 19 comments of Sandesh Kumar

@jed326 > If the test depends on a specific number of segments then can we [1] add an assertion for the segment count and [2] force merge to the correct...

> > The failures in flaky tests reported in different CI calls can only occur when there are multiple segments so this is more of a calculated change that we...

> Can you explicitly issue IndexWriter.commit to create multiple segments and confirm the issue? Just want to ensure that this change is helping Hey @jainankitk, thanks for suggesting this. In...

I think I missed where I can check the number of segments using `reader.leaves.size()` > Since you have an `IndexReader`, could `expectedCollectCount` be a `Function`, then for the case where...

@msfroh I see this is merged. Would you recommend reverting this back and have a multiple segment case as we discussed offline. Tbh, I like it in present state since...

As per offline discussion with @msfroh - will keep this change and will add a separate test to cover multi-segment scenario.

[Search Triage] @sejli Is there any further/ongoing work planned on this? If not, can we close this issue?

[Search Triage] @sejli Do you plan to take a look on this?

[Search Triage] @frankschlehofer Hey, Are you still working on this?