Wrong assertion in BatchFetchQueueHelper

Description

There is an assertion in the removeNotFoundBatchLoadableEntityKeys method of the BatchFetchQueueHelper class that seems to be wrong[1]. This error can make test cases fail unexpectedly.

When reaching this method having the results list with a size lower than the ids array (Because the ids array has duplicated values: this can happen when using the padding fetch style), then the assertion can fail.

I think that the ids.length is not the size to check in the assertion. Attached patch with the proposed solution.

1- BatchFetchQueueHelper Wrong Assertion

Environment

None

Status

Assignee

Guillaume Smet

Reporter

Carlos Aristu

Fix versions

Labels

None

backPortable

None

Suitable for new contributors

None

Requires Release Note

None

Pull Request

None

backportDecision

None

Components

Affects versions

5.3.2

Priority

Major