Fixed couple of "leaks" when gc is disabled#979
Fixed couple of "leaks" when gc is disabled#979Mephius wants to merge 12 commits intodpkp:masterfrom Mephius:master
Conversation
Since I'm using kafka-python in an environment with disabled gc these "leaks" hit me hard.
| del X | ||
|
|
||
| # Don't del it here, cause with gc disabled this "leaks" to garbage | ||
| # del X |
There was a problem hiding this comment.
this one is a vendored library, and I really would prefer fixing upstream. otherwise I'm likely to forget and overwrite when I import a newer version. Have you checked if this is fixed already in six?
There was a problem hiding this comment.
Upstream doesn't have this fix, but I've filed a pull request: benjaminp/six#176
|
|
||
|
|
||
| class BaseSelector(six.with_metaclass(ABCMeta)): | ||
| @six.add_metaclass(ABCMeta) |
There was a problem hiding this comment.
same re: vendored library. Can you check with upstream version?
There was a problem hiding this comment.
Pull request to vendored library: berkerpeksag/selectors34#7
There was a problem hiding this comment.
Upstream merged pull request.
…eter to cleanup WekMethod target preliminary
|
Sorry for spamming PR with commits. Python is magic sometimes. |
|
I merged the selectors34 code from upstream separately. I am ok merging the rest, though I would prefer to wait for upstream to merge before changing six. That said, given that the change is mostly trivial I would be ok merging. Can you rebase? |
|
Merged manually as |
|
Thank you! |
Bump `six` to `1.11.0`. Most changes do not affect us, but it's good to stay up to date. Also, we will likely start vendoring `enum34` in which case benjaminp/six#178 is needed. Note that this preserves the `kafka-python` customization from #979 which has been submitted upstream as benjaminp/six#176 but not yet merged.
Bump `six` to `1.11.0`. Most changes do not affect us, but it's good to stay up to date. Also, we will likely start vendoring `enum34` in which case benjaminp/six#178 is needed. Note that this preserves the `kafka-python` customization from #979 which has been submitted upstream as benjaminp/six#176 but not yet merged.
Bump `six` to `1.11.0`. Most changes do not affect us, but it's good to stay up to date. Also, we will likely start vendoring `enum34` in which case benjaminp/six#178 is needed. Note that this preserves the `kafka-python` customization from #979 which has been submitted upstream as benjaminp/six#176 but not yet merged.
Bump `six` to `1.11.0`. Most changes do not affect us, but it's good to stay up to date. Also, we will likely start vendoring `enum34` in which case benjaminp/six#178 is needed. Note that this preserves the `kafka-python` customization from dpkp/kafka-python#979 which has been submitted upstream as benjaminp/six#176 but not yet merged.
2 participants
Since I'm using kafka-python in an environment with disabled gc these
"leaks" hit me hard.