Reduce StorageSubscription size

[ltrzesniewski]

From @ocoanet on January 13, 2017 10:49

The StorageSubscription size can be quite large for peers with many subscriptions. The subscription binding keys are serialized in StorageConvertionExtensions.SerializeBindingKeys. This method could be optimized for the very common scenario where binding key parts are numeric values.

For example:

private static void SerializeBindingKey(BinaryWriter binaryWriter, BindingKey bindingKey)
{
    var numericParts = new List<int>(bindingKey.PartCount);
    for (var partIndex = 0; partIndex < bindingKey.PartCount; partIndex++)
    {
        int numericValue;
        if (!int.TryParse(bindingKey.GetPart(partIndex), out numericValue))
            break;

        numericParts.Add(numericValue);
        if (numericParts.Count == bindingKey.PartCount)
        {
            binaryWriter.Write(bindingKey.PartCount | _isNumeric);
            numericParts.ForEach(binaryWriter.Write);
            return;
        }
    }

    binaryWriter.Write(bindingKey.PartCount);
    for (var partIndex = 0; partIndex < bindingKey.PartCount; partIndex++)
    {
        binaryWriter.Write(bindingKey.GetPart(partIndex));
    }
}

Of course, a test should be writen to measure the improvements of this change.

Copied from original issue: Abc-Arbitrage/Zebus.Directory#10