gortiz opened a new pull request, #10492: URL: https://github.com/apache/pinot/pull/10492
Trivial fix on a warning detected by static checker Older code was: ``` long dictionarySize = info.getDistinctValueCount() * spec.getDataType().size(); ``` This could produce an overflow if the number of distinct values is close to Integer.MAX_INT / 4, which seems rare, but possible. New code just cast one of the members as long in order to be sure that 64bit product is used. -- 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