You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Allocations are all from PooledByteBufferWriter. I think we basically doubled the size of it by adding a Stream field to it, and making it inherit from PipeWriter which has a private Stream field as well. So it went from int and byte[], to int, byte[], Stream, Stream.
Activity
BrennanConroy commentedon Jun 3, 2024
Narrowed this down to dotnet/runtime@8f2f3da...010a744
Which is the Json change that shouldn't really affect too much yet 😆
sebastienros commentedon Jun 3, 2024
But it's also regression workingset/allocations, is that expected? #1994
BrennanConroy commentedon Jun 3, 2024
Haven't looked at that one yet.
BrennanConroy commentedon Jun 3, 2024
Allocations are all from
PooledByteBufferWriter
. I think we basically doubled the size of it by adding aStream
field to it, and making it inherit fromPipeWriter
which has a privateStream
field as well. So it went fromint
andbyte[]
, toint
,byte[]
,Stream
,Stream
.