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
{{ message }}
This repository has been archived by the owner on May 13, 2022. It is now read-only.
It seems that System.Data.HashFunction.CityHash NuGet package is outdated. We had a performance test and the repository's version of CityHash128 (from the current master branch) is very good.
But the NuGet package's version of CityHash128 is very bad. It's even slower than MD5 and also consumes high memory amount:
I used the benchmark source code below to compare the performance and memory usage:
using System;
using System.Data.HashFunction.CityHash;
using System.Security.Cryptography;
using BenchmarkDotNet.Attributes;
using BenchmarkDotNet.Running;
namespace TestCityHash
{
class Program
{
static void Main(string[] args)
{
var summary = BenchmarkRunner.Run<CityHash128vs64vs32>();
}
}
[ShortRunJob]
[MemoryDiagnoser]
public class CityHash128vs64vs32
{
private readonly byte[] _data = new byte[1000];
private readonly MD5 _md5 = MD5.Create();
private readonly ICityHash _cityHash128 = HashFactory.CreateCityHash128();
private readonly ICityHash _cityHash64 = HashFactory.CreateCityHash64();
private readonly ICityHash _cityHash32 = HashFactory.CreateCityHash32();
public CityHash128vs64vs32()
{
new Random().NextBytes(_data);
}
[Benchmark(Baseline = true)]
public byte[] Md5() => _md5.ComputeHash(_data);
[Benchmark]
public byte[] CityHash128() => _cityHash128.ComputeHash(_data).Hash;
[Benchmark]
public byte[] CityHash64() => _cityHash64.ComputeHash(_data).Hash;
[Benchmark]
public byte[] CityHash32() => _cityHash32.ComputeHash(_data).Hash;
}
public class HashFactory
{
public static ICityHash CreateCityHash128()
{
var config = new CityHashConfig
{
HashSizeInBits = 128
};
return CityHashFactory.Instance.Create(config);
}
public static ICityHash CreateCityHash64()
{
var config = new CityHashConfig
{
HashSizeInBits = 64
};
return CityHashFactory.Instance.Create(config);
}
public static ICityHash CreateCityHash32()
{
var config = new CityHashConfig
{
HashSizeInBits = 32
};
return CityHashFactory.Instance.Create(config);
}
}
}
I have compared the source code (package version vs branch version) and have to conclude that the CityHash128 implementation from the NuGet package is definitely outdated. Maybe it is possible to publish a new version of the NuGet package that contains implementation from the current master branch? E.g. from v2.0.0 to 2.0.1. That would be great. Thank you!
The text was updated successfully, but these errors were encountered:
It seems that System.Data.HashFunction.CityHash NuGet package is outdated. We had a performance test and the repository's version of CityHash128 (from the current master branch) is very good.
But the NuGet package's version of CityHash128 is very bad. It's even slower than MD5 and also consumes high memory amount:
I used the benchmark source code below to compare the performance and memory usage:
I have compared the source code (package version vs branch version) and have to conclude that the CityHash128 implementation from the NuGet package is definitely outdated. Maybe it is possible to publish a new version of the NuGet package that contains implementation from the current master branch? E.g. from v2.0.0 to 2.0.1. That would be great. Thank you!
The text was updated successfully, but these errors were encountered: