scylladb / cpp-rust-driver

API-compatible rewrite of https://github.com/scylladb/cpp-driver as a wrapper for Rust driver.
GNU Lesser General Public License v2.1
11 stars 11 forks source link

Fix memory leak error afer adding schema metadata #95

Closed Gor027 closed 1 year ago

Gor027 commented 1 year ago

Build and Cassandra workflows did not report definitely lost memory leaks as errors and support for fetching schema metadata was merged with memory leaks.

Fixed memory leaking error and added appropriate options to valgrind to report only definitely lost memory as error.

Pre-review checklist

Gor027 commented 1 year ago

@Lorak-mmk While debugging and testing #76 to prevent memory leaks I found that the schema metadata related PR #72 was merged still having some memory leaks that the GitHub Actions workflows did not detect and report. I hope this PR will prevent such cases in the future.