search

fogbow / fogbow-mono-manager

Other
6 stars 3 forks source link

Close connection in some AccountingDataStore methods #343

Closed llvieira closed 4 years ago

llvieira commented 6 years ago

Closing connections and also fixing some typos. Fixes #328.

paulofelipefeitosa commented 6 years ago

Hello @llvieira ! Did you run the tests? Looks good to me.

llvieira commented 6 years ago

@paulofelipefeitosa I ran the tests and what happens was a non-deterministic behavior, however I was surprised by the fact that running each test particularly it works well.

Anyway, thanks for reviewing it! :smiley:

paulofelipefeitosa commented 6 years ago

Hello @llvieira, this issue is being investigated by @ordansantos.