{"action":"created","issue":{"url":"https://api.github.com/repos/spring-projects/spring-integration/issues/1848","repository_url":"https://api.github.com/repos/spring-projects/spring-integration","labels_url":"https://api.github.com/repos/spring-projects/spring-integration/issues/1848/labels{/name}","comments_url":"https://api.github.com/repos/spring-projects/spring-integration/issues/1848/comments","events_url":"https://api.github.com/repos/spring-projects/spring-integration/issues/1848/events","html_url":"https://github.com/spring-projects/spring-integration/pull/1848","id":165072688,"number":1848,"title":"INT-4069: Use AnnotatedElementUtils","user":{"login":"ghchapman","id":453234,"avatar_url":"https://avatars.githubusercontent.com/u/453234?v=3","gravatar_id":"","url":"https://api.github.com/users/ghchapman","html_url":"https://github.com/ghchapman","followers_url":"https://api.github.com/users/ghchapman/followers","following_url":"https://api.github.com/users/ghchapman/following{/other_user}","gists_url":"https://api.github.com/users/ghchapman/gists{/gist_id}","starred_url":"https://api.github.com/users/ghchapman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ghchapman/subscriptions","organizations_url":"https://api.github.com/users/ghchapman/orgs","repos_url":"https://api.github.com/users/ghchapman/repos","events_url":"https://api.github.com/users/ghchapman/events{/privacy}","received_events_url":"https://api.github.com/users/ghchapman/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2016-07-12T12:53:15Z","updated_at":"2016-07-12T14:59:15Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/spring-projects/spring-integration/pulls/1848","html_url":"https://github.com/spring-projects/spring-integration/pull/1848","diff_url":"https://github.com/spring-projects/spring-integration/pull/1848.diff","patch_url":"https://github.com/spring-projects/spring-integration/pull/1848.patch"},"body":"JIRA: https://jira.spring.io/browse/INT-4069\r\n\r\nFinding annotations with AnnotatedElementUtils allows us to use\r\nexplicit annotation attribute overrides specified by the @AliasFor\r\nannotation."},"comment":{"url":"https://api.github.com/repos/spring-projects/spring-integration/issues/comments/232074897","html_url":"https://github.com/spring-projects/spring-integration/pull/1848#issuecomment-232074897","issue_url":"https://api.github.com/repos/spring-projects/spring-integration/issues/1848","id":232074897,"user":{"login":"artembilan","id":1071590,"avatar_url":"https://avatars.githubusercontent.com/u/1071590?v=3","gravatar_id":"","url":"https://api.github.com/users/artembilan","html_url":"https://github.com/artembilan","followers_url":"https://api.github.com/users/artembilan/followers","following_url":"https://api.github.com/users/artembilan/following{/other_user}","gists_url":"https://api.github.com/users/artembilan/gists{/gist_id}","starred_url":"https://api.github.com/users/artembilan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/artembilan/subscriptions","organizations_url":"https://api.github.com/users/artembilan/orgs","repos_url":"https://api.github.com/users/artembilan/repos","events_url":"https://api.github.com/users/artembilan/events{/privacy}","received_events_url":"https://api.github.com/users/artembilan/received_events","type":"User","site_admin":false},"created_at":"2016-07-12T14:59:15Z","updated_at":"2016-07-12T14:59:15Z","body":"Well, we can go any path. The only problem I worry about, if it is really reasonable to merge this minor improvement into the current point release. According our previous private discussion this looks more like academic investigation from your side and don't have any immediate requirement from end-user perspective. (Correct me if I'm wrong). \r\n\r\nOn the other hand after some functional analysis of the target `AnnotatedElementUtils.findMergedAnnotation()` premize, we may even decide to reject this `MethodAnnotationPublisherMetadataSource`, but apply some other one.\r\n\r\nThat's why I'm asking to make some investigation and convince us with some arguments that exactly this should be changed not that one. \r\n\r\nAs I said: the PR is fine, but I can't merge it as is with more background. My intention to comeback to it during the proper deadline to have more clean head for better investigation. That's one more \"why\" I'd prefer do not merge it to current release.\r\n"}}
{"actor":{"display_login":"artembilan"}}