dnickless

Error when calling FastThreadLocal.RemoveAll(): Collection was modified

1

https://github.com/Azure/DotNetty/blob/47f5ec7303037f1360615d182939e04d8619a2b3/src/DotNetty.Common/FastThreadLocal.cs#L32 I think that comment is unpleasantly true, you will need to create a clone as you're removing items from the collection that you're iterating over... This is what I'm...

CSHARP-4292: Performance: Eliminated allocations in GetHashCode methods

1

Eliminated some boxing problems leading to substantial allocations in our code. Details located here: https://jira.mongodb.org/browse/CSHARP-4292

Performance/(de-)serialization: Added two caches to avoid reflection calls

1

`TypeNameDiscriminator` is a static helper class that kicks in during serialization and deserialization when using type names as discriminators. There currently is quite some reflection and string-fiddling inside two crucial...

Strategy pattern for BsonClassMapSerializer

1

The `BsonClassMapSerializer.DeserializeClass(BsonDeserializationContext context)` method contained code to handle two basic cases depending on the value of `_classMap.HasCreatorMaps`: The deserialization using a `Dictionary` and the "standard" deserialization. By introducing a strategy...

There was some reflection code in the `Deserialize` method which would get called repeatedly. But since the ClassMap is frozen (gets checked in the constructor) it should be sufficient to...

CSHARP-1972: Further performance optimization of the Hex conversion methods in BsonUtils

1

May I suggest another performance optimization around the hex string to byte array (and back) conversion methods in `BsonUtils`. This change introduces some precalculated jump tables that cost a tiny...