Merge pull request #26 from stackify/feature/SF-6393-Millisecond-Ordering

sporty81 · web-flow · commit 66476d3fee6b · 2017-02-16T11:35:04.000-06:00
SF-6393 Use 1 as the initial millisecond order
diff --git a/Src/StackifyLib/Internal/Logs/LogClient.cs b/Src/StackifyLib/Internal/Logs/LogClient.cs
@@ -127,7 +127,7 @@ public bool ErrorShouldBeSent(StackifyError error)
 
 
         private static long _lastEpochMs = 0;
-        private static int _millisecondCount = 0;
+        private static int _millisecondCount = 1;
 
         public void QueueMessage(LogMsg msg)
         {
@@ -151,8 +151,8 @@ public void QueueMessage(LogMsg msg)
                 // reset counter if we are no longer in the same ms
                 //https://msdn.microsoft.com/en-us/library/system.threading.interlocked_methods(v=vs.110).aspx
                 Interlocked.Exchange(ref _lastEpochMs, msg.EpochMs);
-                Interlocked.Exchange(ref _millisecondCount, 0);
-                msg.Order = 0;
+                Interlocked.Exchange(ref _millisecondCount, 1);
+                msg.Order = 1;
             }
             else if (_lastEpochMs == msg.EpochMs)
             {

Original file line number	Diff line number	Diff line change
`@@ -127,7 +127,7 @@ public bool ErrorShouldBeSent(StackifyError error)`
`127`	`127`
`128`	`128`
`129`	`129`	`private static long _lastEpochMs = 0;`
`130`		`- private static int _millisecondCount = 0;`
	`130`	`+ private static int _millisecondCount = 1;`
`131`	`131`
`132`	`132`	`public void QueueMessage(LogMsg msg)`
`133`	`133`	`{`
`@@ -151,8 +151,8 @@ public void QueueMessage(LogMsg msg)`
`151`	`151`	`// reset counter if we are no longer in the same ms`
`152`	`152`	`//https://msdn.microsoft.com/en-us/library/system.threading.interlocked_methods(v=vs.110).aspx`
`153`	`153`	`Interlocked.Exchange(ref _lastEpochMs, msg.EpochMs);`
`154`		`- Interlocked.Exchange(ref _millisecondCount, 0);`
`155`		`- msg.Order = 0;`
	`154`	`+ Interlocked.Exchange(ref _millisecondCount, 1);`
	`155`	`+ msg.Order = 1;`
`156`	`156`	`}`
`157`	`157`	`else if (_lastEpochMs == msg.EpochMs)`
`158`	`158`	`{`