aerospike / aerospike-client-csharp

Aerospike C# Client Library
70 stars 47 forks source link

Use List instead of Array. #54

Closed farmerau closed 4 years ago

farmerau commented 4 years ago

Sometimes, when ExecuteBatchRetry is called, orig is not present in either commands or cmds. We had two options:

  1. Check to see if either array contains the object and then create the array with the correct size (expensive)
  2. Use a list, and then convert it to an array when finished (less expensive).,

This prevents an IndexOutOfRangeException Seen in issue #50 .

This has been tested locally in my application, with forced timeouts by using clumsy 0.2 for Windows. With these changes, we're able to achieve all iterations in our async batch read.