Int 32 overflow on geo centroid aggregation

[angeloprobst]

elasticsearch/server/src/main/java/org/elasticsearch/search/aggregations/metrics/geocentroid/InternalGeoCentroid.java

Line 109 in 6f244eb

int totalCount = 0;

We have a Elasticsearch with billions of docs with geopoint and this local variable (int totalCount) is being overflowed when a bucket has more than 2.147Bi docs.

[elasticmachine]

Pinging @elastic/es-analytics-geo (:Analytics/Geo)

[talevy]

Hi @angeloprobst! thank you so much for reporting this issue. Terribly sorry you ran into this as your reduced so many points! The issue should be resolved in the versions listed as labels in #56370.

[angeloprobst]

Hi guys. Any chance to have this fix applied to ES 6.x.x series? Thanks. Regards, Angelo

…

On Thu, May 7, 2020, 18:32 Tal Levy ***@***.***> wrote: Hi @angeloprobst <https://github.com/angeloprobst>! thank you so much for reporting this issue. Terribly sorry you ran into this as your reduced so many points! The issue should be resolved in the versions listed as labels in #56370 <#56370>. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#55992 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AICWAWJUJW6R6YOXADKNFBDRQMSH3ANCNFSM4MVCTMFQ> .

[talevy]

Hi @angeloprobst thanks for bringing that up. 6.8.x is unfortunately in a maintenance mode where we would likely only deliver a new patch release for higher impact items than this. Sorry to say that. Let me know if you'd like some assistance in getting prepared to upgrade to the next released version with this bug fix, I'd be happy to help!