Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Fix memory management #131

Merged
merged 7 commits into from
Apr 20, 2023
Merged

fix: Fix memory management #131

merged 7 commits into from
Apr 20, 2023

Conversation

hermanschaaf
Copy link
Member

@hermanschaaf hermanschaaf commented Apr 19, 2023

A number of fixes, but most notably removing support for Retain/Release as we've decided to only support the default Go memory allocator going forward. With the Go allocator we can rely on Go's garbage collector.

@github-actions github-actions bot added the fix label Apr 19, 2023
@github-actions github-actions bot added fix and removed fix labels Apr 20, 2023
@hermanschaaf hermanschaaf added the automerge Add to automerge PRs once requirements are met label Apr 20, 2023
@kodiakhq kodiakhq bot merged commit 9b97bae into main Apr 20, 2023
@kodiakhq kodiakhq bot deleted the memory-fixes branch April 20, 2023 08:27
kodiakhq bot pushed a commit that referenced this pull request Apr 20, 2023
🤖 I have created a release *beep* *boop*
---


## [2.0.2](v2.0.1...v2.0.2) (2023-04-20)


### Bug Fixes

* **deps:** Update module github.com/cloudquery/plugin-sdk/v2 to v2.3.3 ([#127](#127)) ([7117ddf](7117ddf))
* **deps:** Update module github.com/cloudquery/plugin-sdk/v2 to v2.3.4 ([#129](#129)) ([19b35be](19b35be))
* **deps:** Update module github.com/cloudquery/plugin-sdk/v2 to v2.3.5 ([#130](#130)) ([b45fa1f](b45fa1f))
* **deps:** Update module github.com/cloudquery/plugin-sdk/v2 to v2.3.6 ([#132](#132)) ([2157205](2157205))
* Fix memory management ([#131](#131)) ([9b97bae](9b97bae))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Add to automerge PRs once requirements are met fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants