apache / mahout

Mirror of Apache Mahout
Apache License 2.0
2.14k stars 945 forks source link

[Mahout-1958] Change the variable type to long to avoid overflow #444

Open balashashanka opened 7 months ago

balashashanka commented 7 months ago

Purpose of PR:

Please give a short description of what this PR is for.

Important ToDos

Please mark each with an "x"

If all of these things aren't complete, but you still feel it is appropriate to open a PR, please add [WIP] after MAHOUT-XXXX before the descriptions- e.g. "MAHOUT-XXXX [WIP] Description of Change"

Does this change break earlier versions?

Is this the beginning of a larger project for which a feature branch should be made?

rawkintrevo commented 7 months ago

Thanks for the contrib @balashashanka

Successfully built and ran all unit tests, verified that all tests pass locally.

Did mvn clean install build without failures?

andrewmusselman commented 7 months ago

We can merge if this is looking good @balashashanka , did it build for you?