-
Notifications
You must be signed in to change notification settings - Fork 68
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.
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
Don't force the use of a specific DB index #39
Comments
can we benchmark with and without? |
Reading @notahat's comments in the bug report. It doesn't sound deterministic. Perhaps putting a flip around this is a way forward. Move forward with the change, while allowing a fast way to revert the behaviour if everything goes wrong. |
This is not an urgent fix. The index requested is still the appropriate index to use. We don't gain much from doing this work. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We include a work around for a MySQL query optimiser defect
https://github.com/envato/double_entry/blob/master/lib/double_entry/balance_calculator.rb#L42-L48
https://github.com/envato/double_entry/blob/master/lib/double_entry/validation/line_check.rb#L55-L63
This MySQL defect has been patched (in 2010). Is it time to remove the work around?
The text was updated successfully, but these errors were encountered: