Skip to content

[BUG] Add Unit Test for Cardinality Aggregation for different scorer paths #18190

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

Open
expani opened this issue May 3, 2025 · 0 comments
Open
Labels
bug Something isn't working Search Search query, autocomplete ...etc untriaged

Comments

@expani
Copy link
Contributor

expani commented May 3, 2025

Describe the bug

While upgrading to Lucene 10.2.1, we observed an exception with cardinality-agg-low/high operation in Big5 workload.

See stack trace #17961 (comment)

Since, the query for cardinality-agg-low/high is MatchAllDocsQuery -> which produces a -> ConstantScoreWeight -> creates a -> ConstantScorerSupplier -> creates a -> DenseConjunctionBulkScorer for some cases unlike DefaultBulkScorer in all cases in older versions.

This change was introduced in Lucene 10.2.0 with https://github.com/apache/lucene/pull/14359/files

We need to add a unit test which covers the same and any other assumption which might not hold true after upgrade.

cc: @bowenlan-amzn

Related component

Search

To Reproduce

#17961 (comment)

Expected behavior

Unit test should catch this case instead of showing up in Big5 workload

Additional Details

No response

@expani expani added bug Something isn't working untriaged labels May 3, 2025
@github-actions github-actions bot added the Search Search query, autocomplete ...etc label May 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Search Search query, autocomplete ...etc untriaged
Projects
Status: 🆕 New
Development

No branches or pull requests

1 participant