shenyu0127 opened a new pull request, #11022: URL: https://github.com/apache/pinot/pull/11022
The test is flaky because the query uses the default `LIMIT` of 10 and there is a race condition in [`ConcurrentIndexedTable::upsertWithoutOrderBy`](https://github.com/apache/pinot/blob/f3068bc93165e359cc3d846c40f62312ecc98af3/pinot-core/src/main/java/org/apache/pinot/core/data/table/ConcurrentIndexedTable.java#L73) which might return more than `LIMIT` number of rows. The race condition will not produce wrong results because in the broker there is a trim logic. The test gets the result from the server. Tested using `invocationCount = 200`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
