search

eBay / go-ovn

A Go library for OVN Northbound/Southbound DB access using native OVSDB protocol
Apache License 2.0
108 stars 59 forks source link

client.Close() should handle cache release and remove the disconnect callback. #89

Open hzhou8 opened 4 years ago

hzhou8 commented 4 years ago

Maybe this API is not needed in most situation. However, current implementation has problems:

1) It doesn't cleanup resources and doesn't preventing the other APIs to continue reading the data. The expected behavior should be cleanup all the cache and all the later API calls for read/write data should fail.

2) Currently we have auto-reconnect supported. This API need to reset the disconnect callback so that it won't automatically reconnect by itself as a result of Close().