-
Notifications
You must be signed in to change notification settings - Fork 6
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
.list processing & DB speedups #619
Conversation
Squash 85b2a32, 794a0be, 90f08a1 * Nix HighwaySegment::clin_mtx; use Route::mtx * Lock/unlock once per store_traveled_segments call * Consolidate lock/unlock ops w/calls to mark_label(s)_in_use
Squash 41fdc3a (cp 39db917), 4b4813e (cp f968c5b), 1d34eba * get_trim_line() only as needed; remember it * std::string fields * Nix orig_line; use intact lines[l] * Fix bug that flagged tab delimiters as invalid chars in userlog errors for invalid fields
44d62f0 ReadListThread mutex efficiencyBefore: After: RAM:
Secondary benefits: |
f5a4e85 reduce string construction3 problems:
3 solutions:
Bugfix: |
Ready to merge, or is more on the way? |
Ready to merge. |
I do have a couple more commits on the way now. |
DB table speedupswaypoints (5e8ebb8)First, let's get rid of The only change to the
segmentsphase 1 (5e8ebb8)
phase 2 (0ff0af1)When constructing clinched (0ff0af1)This table takes 1.5-2x as long to write now, but that's the price we pay for greatly speeding up the
segments + clinched combined time |
Ready to merge. |
will use this for tonight's site update, thanks @yakra as always for all the work on this! |
@yakra when you get the chance, can ya check your PM's on the forum. Thanks. :) |
Haha, been thinking about how I need to get back there soon. |
hehehe. Some people over there think you've passed along to the next life already. :P (j/k of course) |
Closes yakra#253
Closes #466
Closes yakra#278
Closes yakra#213