You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@lucenenet.apache.org by GitBox <gi...@apache.org> on 2022/02/03 09:03:34 UTC

[GitHub] [lucenenet] NightOwl888 commented on pull request #613: Factored out WeakDictionary in favor of weak events using Prism.Core

NightOwl888 commented on pull request #613:
URL: https://github.com/apache/lucenenet/pull/613#issuecomment-1028758427


   @eladmarg 
   
   Not sure. Being that `ConditionalWeakTable` requires the key and values to be dereferenced before it will clean them up, this poses a problem to be able to check whether they actually still exist (because they cannot be looked up without those references). So, maybe the only way to check is to profile/benchmark the app to see if it makes a difference, but other suggestions are welcome.
   
   However, testing on .NET Framework seems to have settled at around 1-2 min longer than .NET 5, where before it varied and was for the most part around 14 minutes instead of around 11-12 minutes.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@lucenenet.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org