PowerDNS / lightningstream

Lightning Stream syncs LMDB databases through S3 buckets between multiple servers, including PowerDNS Authoritative server 4.8+ LMDBs
https://doc.powerdns.com/lightningstream/
MIT License
25 stars 15 forks source link

Upgrade `simpleblob` to v0.2.4 #59

Closed ahouene closed 1 year ago

ahouene commented 1 year ago

simpleblob@v0.2.4 introduces two bug fixes for the S3 backend.

Closes #55 #57

From the release page:

This fixes a resource leak that can cause growing memory and CPU usage over time (https://github.com/PowerDNS/simpleblob/issues/44) for all S3 users.

It also fixes the update marker not working when a global S3 prefix is used, which causes simpleblob to use more expensive LIST API calls, leading to increased S3 usage costs. What's Changed

S3 backend: fix resource leak due to missing Close for GetObject by @ahouene in https://github.com/PowerDNS/simpleblob/pull/45 (fixes https://github.com/PowerDNS/simpleblob/issues/44)
Fix global prefix not working with update marker by @ahouene in https://github.com/PowerDNS/simpleblob/pull/42 (fixes https://github.com/PowerDNS/simpleblob/issues/40)
ahouene commented 1 year ago

Closing this one in favor of #58 who was faster than me!