{"action":"created","issue":{"url":"https://api.github.com/repos/dropwizard/dropwizard/issues/1959","repository_url":"https://api.github.com/repos/dropwizard/dropwizard","labels_url":"https://api.github.com/repos/dropwizard/dropwizard/issues/1959/labels{/name}","comments_url":"https://api.github.com/repos/dropwizard/dropwizard/issues/1959/comments","events_url":"https://api.github.com/repos/dropwizard/dropwizard/issues/1959/events","html_url":"https://github.com/dropwizard/dropwizard/pull/1959","id":213705494,"number":1959,"title":"Support `@UnitOfWork` in sub-resources","user":{"login":"arteam","id":1717632,"avatar_url":"https://avatars3.githubusercontent.com/u/1717632?v=3","gravatar_id":"","url":"https://api.github.com/users/arteam","html_url":"https://github.com/arteam","followers_url":"https://api.github.com/users/arteam/followers","following_url":"https://api.github.com/users/arteam/following{/other_user}","gists_url":"https://api.github.com/users/arteam/gists{/gist_id}","starred_url":"https://api.github.com/users/arteam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/arteam/subscriptions","organizations_url":"https://api.github.com/users/arteam/orgs","repos_url":"https://api.github.com/users/arteam/repos","events_url":"https://api.github.com/users/arteam/events{/privacy}","received_events_url":"https://api.github.com/users/arteam/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-03-13T08:55:10Z","updated_at":"2017-03-13T09:17:38Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/dropwizard/dropwizard/pulls/1959","html_url":"https://github.com/dropwizard/dropwizard/pull/1959","diff_url":"https://github.com/dropwizard/dropwizard/pull/1959.diff","patch_url":"https://github.com/dropwizard/dropwizard/pull/1959.patch"},"body":"Currently Dropwizard doesn't open transactions in sub-resources as reported in #1806. The problem is that Dropwizard scans resource methods for `@UnitOfWork` during resource initialization and sub-resource methods are not resolved at that time and can't be registered by Dropwizard. The fix is\r\nto defer the lookup of the `@UnitOfWork` annotation on a method until it's invoked and than cache it. One downside of this approache is that `ConcurrentMap` doesn't support null values, so we have to use `Optional` from `UnitOfWork` which can add an additional overhead, but it should be negligible."},"comment":{"url":"https://api.github.com/repos/dropwizard/dropwizard/issues/comments/286052854","html_url":"https://github.com/dropwizard/dropwizard/pull/1959#issuecomment-286052854","issue_url":"https://api.github.com/repos/dropwizard/dropwizard/issues/1959","id":286052854,"user":{"login":"coveralls","id":2354108,"avatar_url":"https://avatars2.githubusercontent.com/u/2354108?v=3","gravatar_id":"","url":"https://api.github.com/users/coveralls","html_url":"https://github.com/coveralls","followers_url":"https://api.github.com/users/coveralls/followers","following_url":"https://api.github.com/users/coveralls/following{/other_user}","gists_url":"https://api.github.com/users/coveralls/gists{/gist_id}","starred_url":"https://api.github.com/users/coveralls/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/coveralls/subscriptions","organizations_url":"https://api.github.com/users/coveralls/orgs","repos_url":"https://api.github.com/users/coveralls/repos","events_url":"https://api.github.com/users/coveralls/events{/privacy}","received_events_url":"https://api.github.com/users/coveralls/received_events","type":"User","site_admin":false},"created_at":"2017-03-13T09:17:38Z","updated_at":"2017-03-13T09:17:38Z","body":"\n[![Coverage Status](https://coveralls.io/builds/10560157/badge)](https://coveralls.io/builds/10560157)\n\nCoverage increased (+0.07%) to 84.754% when pulling **015a19438c4b1752c2d1d585c3e73762a89b691a on fix_sublocator_bug** into **cc84e68ba967ec8013405e3c284dfff7c05aa6cd on release/1.1.x**.\n"}}
{"actor":{"display_login":"coveralls"}}