aerospike / aerospike-client-go

Aerospike Client Go
Apache License 2.0
430 stars 199 forks source link

ExecuteTask.OnComplete function look weird without any comment for reason. #435

Open namkazt opened 1 month ago

namkazt commented 1 month ago

@khaf please take a look on this

image

that last part look weird and it do not have any comment for a reason why.

khaf commented 1 month ago

Thanks for your feedback. This seems to have been the result of a botched merge, although it had no effect on the correctness of the code (they were unreachable code), but I thought I removed it for the v7.3 or v7.4 release. Which version of the code is this from?

namkazt commented 1 month ago

oh, i'm using v7.1.0, thanks i will check newer version.