mahout icon indicating copy to clipboard operation
mahout copied to clipboard

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

Open balashashanka opened this issue 1 year ago • 2 comments

Purpose of PR:

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

Important ToDos

Please mark each with an "x"

  • [x] A JIRA ticket exists (if not, please create this first)[https://issues.apache.org/jira/browse/mahout/]
  • [x] Title of PR is "MAHOUT-XXXX Brief Description of Changes" where XXXX is the JIRA number.
  • [ ] Created unit tests where appropriate
  • [ ] Added licenses correct on newly added files
  • [x] Assigned JIRA to self
  • [ ] Added documentation in scala docs/java docs, and to website
  • [ ] Successfully built and ran all unit tests, verified that all tests pass locally.

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?

balashashanka avatar Apr 19 '24 15:04 balashashanka

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?

rawkintrevo avatar Apr 23 '24 20:04 rawkintrevo

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

andrewmusselman avatar May 03 '24 17:05 andrewmusselman