Title: Possible leak in double-booking cursor · Issue #31 · gitpython-developers/smmap · GitHub
Open Graph Title: Possible leak in double-booking cursor · Issue #31 · gitpython-developers/smmap
X Title: Possible leak in double-booking cursor · Issue #31 · gitpython-developers/smmap
Description: Probably 70fae1f made a double acounting of region-clients in smmap.mman.py#L136, where a "manual" increment_client_count() follows a _obtain_region() which it also increments the client-count. From gitdb TC gitdb.test.test_pack:TestPack...
Open Graph Description: Probably 70fae1f made a double acounting of region-clients in smmap.mman.py#L136, where a "manual" increment_client_count() follows a _obtain_region() which it also increments the client-count. Fro...
X Description: Probably 70fae1f made a double acounting of region-clients in smmap.mman.py#L136, where a "manual" increment_client_count() follows a _obtain_region() which it also increments the client-...
Opengraph URL: https://github.com/gitpython-developers/smmap/issues/31
X: @github
Domain: redirect.github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Possible leak in double-booking cursor","articleBody":"Probably 70fae1f98bb7 made a double acounting of region-clients in [`smmap.mman.py#L136`](https://github.com/gitpython-developers/smmap/blob/master/smmap/mman.py#L136), where a \"manual\" `increment_client_count()` follows a `_obtain_region()` which it also increments the client-count.\n\nFrom _gitdb_ TC `gitdb.test.test_pack:TestPack.test_pack_entity()` I can see that this is indeed the case, where it is increased 2 x 3 times for 3 mmaps, while it is decremented just 3 times.\n","author":{"url":"https://github.com/ankostis","@type":"Person","name":"ankostis"},"datePublished":"2016-10-23T23:57:22.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":8},"url":"https://github.com/31/smmap/issues/31"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:19b88803-04a9-22d6-889b-fd6703f62434 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | 9DA8:2D2FEF:15BF0F8:1D2FF4C:6969A389 |
| html-safe-nonce | 73508d503612363a154cb93139fee25f90cbcef65f67925fb16d7bf29d21ae7f |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI5REE4OjJEMkZFRjoxNUJGMEY4OjFEMkZGNEM6Njk2OUEzODkiLCJ2aXNpdG9yX2lkIjoiMTQ1NjU0NDY1MDE3ODk2MjMxMyIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 278e69d3d1f1490154b30e24346ef4da223e4e15e4ef14afdf46254e6e8a6695 |
| hovercard-subject-tag | issue:184725334 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/gitpython-developers/smmap/31/issue_layout |
| twitter:image | https://opengraph.githubassets.com/010fae9be792a7542a4de460767a499dc5b8dc91828ab690f53163c56da35d87/gitpython-developers/smmap/issues/31 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/010fae9be792a7542a4de460767a499dc5b8dc91828ab690f53163c56da35d87/gitpython-developers/smmap/issues/31 |
| og:image:alt | Probably 70fae1f made a double acounting of region-clients in smmap.mman.py#L136, where a "manual" increment_client_count() follows a _obtain_region() which it also increments the client-count. Fro... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | ankostis |
| hostname | github.com |
| expected-hostname | github.com |
| None | 24c4c97a2d520cb286b35e1a4c22d7a4df3c26a2fa28dd7cdf0e65db327b4de7 |
| turbo-cache-control | no-preview |
| go-import | github.com/gitpython-developers/smmap git https://github.com/gitpython-developers/smmap.git |
| octolytics-dimension-user_id | 503709 |
| octolytics-dimension-user_login | gitpython-developers |
| octolytics-dimension-repository_id | 1865144 |
| octolytics-dimension-repository_nwo | gitpython-developers/smmap |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 1865144 |
| octolytics-dimension-repository_network_root_nwo | gitpython-developers/smmap |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 124667f43168afb6c9c03b7c02eb5b1d2e1be3d9 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width