Open bruno-garcia opened 3 years ago
As Newtonsoft.Json's count is reaching int.MaxValue, this issue was raised: https://github.com/NuGet/NuGetGallery/issues/8476
int.MaxValue
At first glance, we're using long already for the count:
long
https://github.com/dotnet/nuget-trends/blob/dac67d1bd4707a94063b843571127eb055a4cc4f/src/NuGetTrends.Data/DailyDownload.cs#L9
We need to make sure things will work fine once values overflow a signed 32bit integer max value.
As Newtonsoft.Json's count is reaching
int.MaxValue, this issue was raised: https://github.com/NuGet/NuGetGallery/issues/8476At first glance, we're using
longalready for the count:https://github.com/dotnet/nuget-trends/blob/dac67d1bd4707a94063b843571127eb055a4cc4f/src/NuGetTrends.Data/DailyDownload.cs#L9
We need to make sure things will work fine once values overflow a signed 32bit integer max value.