Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Revise the accumulators performance #176

Open
vkostyukov opened this issue Apr 23, 2014 · 2 comments
Open

Revise the accumulators performance #176

vkostyukov opened this issue Apr 23, 2014 · 2 comments
Labels
enhancement
Milestone
la4j-0.6.0

Comments

@vkostyukov
Copy link
Owner

We agreed to use BigDecimal in accumulators. After the micro-becnhamrks enabling we might want to test regular double agains BigDecimal.
@vkostyukov vkostyukov added this to the la4j-0.5.0 milestone Apr 23, 2014
@vkostyukov vkostyukov added bug and removed bug labels Apr 23, 2014
@SamoylovMD
Copy link
Contributor

We can change BigDecimal to double but it may cause loss of precision (if I remember correctly, thats why we used BigDecimal class in current implementation).

@vkostyukov
Copy link
Owner Author

Right. The plan is following:

  • Establish a robust functional testing
  • Establish performance testing with JMH

Then migrate to double if it's possible.

@vkostyukov vkostyukov modified the milestones: la4j-0.6.0, la4j-0.5.0 Jan 5, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
la4j-0.6.0
Development

No branches or pull requests
2 participants
@vkostyukov @SamoylovMD