{"action":"created","issue":{"url":"https://api.github.com/repos/pmem/nvml/issues/841","repository_url":"https://api.github.com/repos/pmem/nvml","labels_url":"https://api.github.com/repos/pmem/nvml/issues/841/labels{/name}","comments_url":"https://api.github.com/repos/pmem/nvml/issues/841/comments","events_url":"https://api.github.com/repos/pmem/nvml/issues/841/events","html_url":"https://github.com/pmem/nvml/pull/841","id":156229576,"number":841,"title":"doc: document interaction with thread cancellation","user":{"login":"marcinslusarz","id":12811376,"avatar_url":"https://avatars.githubusercontent.com/u/12811376?v=3","gravatar_id":"","url":"https://api.github.com/users/marcinslusarz","html_url":"https://github.com/marcinslusarz","followers_url":"https://api.github.com/users/marcinslusarz/followers","following_url":"https://api.github.com/users/marcinslusarz/following{/other_user}","gists_url":"https://api.github.com/users/marcinslusarz/gists{/gist_id}","starred_url":"https://api.github.com/users/marcinslusarz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marcinslusarz/subscriptions","organizations_url":"https://api.github.com/users/marcinslusarz/orgs","repos_url":"https://api.github.com/users/marcinslusarz/repos","events_url":"https://api.github.com/users/marcinslusarz/events{/privacy}","received_events_url":"https://api.github.com/users/marcinslusarz/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-05-23T08:50:37Z","updated_at":"2016-05-23T14:43:29Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/pmem/nvml/pulls/841","html_url":"https://github.com/pmem/nvml/pull/841","diff_url":"https://github.com/pmem/nvml/pull/841.diff","patch_url":"https://github.com/pmem/nvml/pull/841.patch"},"body":"\"Thread cancellation is not supported. The pool will never be corrupted because of canceled thread, but other threads may stall waiting on locks taken by that thread. If  application  wants  to  use pthread_cancel(2),  cancellation  must  be  disabled before calling libpmemobj APIs (see pthread_setcancelstate(2) with PTHREAD_CANCEL_DISABLE).  Deferring cancellation (pthread_setcanceltype with PTHREAD_CANCEL_DEFERRED) is not enough, because libpmemobj internally may call functions that are marked as cancellation points in POSIX API.\"\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"35\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/pmem/nvml/841)\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/pmem/nvml/issues/comments/221000612","html_url":"https://github.com/pmem/nvml/pull/841#issuecomment-221000612","issue_url":"https://api.github.com/repos/pmem/nvml/issues/841","id":221000612,"user":{"login":"peluse","id":4650360,"avatar_url":"https://avatars.githubusercontent.com/u/4650360?v=3","gravatar_id":"","url":"https://api.github.com/users/peluse","html_url":"https://github.com/peluse","followers_url":"https://api.github.com/users/peluse/followers","following_url":"https://api.github.com/users/peluse/following{/other_user}","gists_url":"https://api.github.com/users/peluse/gists{/gist_id}","starred_url":"https://api.github.com/users/peluse/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/peluse/subscriptions","organizations_url":"https://api.github.com/users/peluse/orgs","repos_url":"https://api.github.com/users/peluse/repos","events_url":"https://api.github.com/users/peluse/events{/privacy}","received_events_url":"https://api.github.com/users/peluse/received_events","type":"User","site_admin":false},"created_at":"2016-05-23T14:43:29Z","updated_at":"2016-05-23T14:43:29Z","body":"\n\n\n\n\n\nReviewed 1 of 1 files at r2.\nReview status: all files reviewed at latest revision, all discussions resolved.\n\n---\n\n\n\n*Comments from [Reviewable](https://reviewable.io:443/reviews/pmem/nvml/841)*\n<!-- Sent from Reviewable.io -->\n"}}
{"actor":{"display_login":"peluse"}}