-
Notifications
You must be signed in to change notification settings - Fork 110
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
Faster MREW for DEB ? #62
Comments
The article is great but it doesn't say TLightweightMREW is a clear winner than TMREWSync which is used by DEB? |
@chmichael I agree with @wxinix . From the article we cannot establish a clear winner as also reported in part 3. @chmichael do you have some numbers to highlight the fact the new TLightMREW is much better for this project? |
@spinettaro @wxinix Did you read all the 4 parts ? (i posted only the 1st part, the other is on main page) Do you we have a benchmark for DEB ? |
I did read all parts. It is a great article by a respected Delphi guru. And thank you for referencing those articles. Actually at one time DEB (for my private fork) did use the latest TLightweightMREW, which comes with Sydney 10.4 (in its release note it was noted for its slightly better performance) But when I submitted my PR to the main DEB repo, I sticked to the old MREW, mainly for compatibility with older versions of Delphi, considering TLightweightMREW only works for the latest. We can do a conditional compile of course. But the number in the article shows mixed benchmarking results —— it doesn’t seem to suggest a clear winner. |
I think that only windows can get a better speed but it is safe only with TLightweightMREWx and only in a number of scenarios |
What ever you want guys but in my opinion it will be nice to have that option too! :) |
Hello,
Take a look at the following articles! I think DEB can benefit from this. (it has 4 parts)
https://www.thedelphigeek.com/2020/11/readers-writer-lock-part-1-why.html
Thank you
The text was updated successfully, but these errors were encountered: