TrueCloudLab/frostfs-sdk-csharp
19
0
Fork 4
Code Issues 5 Pull requests Projects Releases Packages Activity Actions

[#20] Client: Optimize memory usage #21

Merged
PavelGrossSpb merged 2 commits from PavelGrossSpb/frostfs-sdk-csharp:misc/memory_optimization into master 2024-09-04 19:51:24 +00:00
Conversation 0 Commits 2 Files changed 46 +2 -2
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 6083834582 - Show all commits

4
src/FrostFS.SDK.ClientV2/Mappers/ContainerId.cs
View file

@ -15,14 +15,14 @@ public static class ContainerIdMapper
public static ContainerID ToMessage(this ContainerId model) public static ContainerID ToMessage(this ContainerId model)
{ {
if (!Cache.Owners.TryGetValue(model, out ContainerID? message)) if (!Cache.Containers.TryGetValue(model, out ContainerID? message))
dstepanov-yadro marked this conversation as resolved Outdated
dstepanov-yadro commented 2024-08-01 14:33:21 +00:00
Outdated
Review
Copy link

Why containerID from owners cache?

Why containerID from owners cache?
PavelGrossSpb commented 2024-08-01 15:04:20 +00:00
Outdated
Review
Copy link

This is a typo, sure. Fixed.

This is a typo, sure. Fixed.
{ {
message = new ContainerID message = new ContainerID
{ {
Value = ByteString.CopyFrom(Base58.Decode(model.Value)) Value = ByteString.CopyFrom(Base58.Decode(model.Value))
}; };
Cache.Owners.Set(model, message, _oneHourExpiration); Cache.Containers.Set(model, message, _oneHourExpiration);
} }
return message!; return message!;