{"action":"closed","issue":{"url":"https://api.github.com/repos/Blazebit/blaze-persistence/issues/356","repository_url":"https://api.github.com/repos/Blazebit/blaze-persistence","labels_url":"https://api.github.com/repos/Blazebit/blaze-persistence/issues/356/labels{/name}","comments_url":"https://api.github.com/repos/Blazebit/blaze-persistence/issues/356/comments","events_url":"https://api.github.com/repos/Blazebit/blaze-persistence/issues/356/events","html_url":"https://github.com/Blazebit/blaze-persistence/issues/356","id":209851199,"number":356,"title":"Try to relax the requirements of @IdMapping","user":{"login":"beikov","id":358916,"avatar_url":"https://avatars3.githubusercontent.com/u/358916?v=3","gravatar_id":"","url":"https://api.github.com/users/beikov","html_url":"https://github.com/beikov","followers_url":"https://api.github.com/users/beikov/followers","following_url":"https://api.github.com/users/beikov/following{/other_user}","gists_url":"https://api.github.com/users/beikov/gists{/gist_id}","starred_url":"https://api.github.com/users/beikov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/beikov/subscriptions","organizations_url":"https://api.github.com/users/beikov/orgs","repos_url":"https://api.github.com/users/beikov/repos","events_url":"https://api.github.com/users/beikov/events{/privacy}","received_events_url":"https://api.github.com/users/beikov/received_events","type":"User","site_admin":false},"labels":[{"id":113480788,"url":"https://api.github.com/repos/Blazebit/blaze-persistence/labels/enhancement","name":"enhancement","color":"84b6eb","default":true},{"id":515512276,"url":"https://api.github.com/repos/Blazebit/blaze-persistence/labels/entity-view","name":"entity-view","color":"006b75","default":false}],"state":"closed","locked":false,"assignee":{"login":"beikov","id":358916,"avatar_url":"https://avatars3.githubusercontent.com/u/358916?v=3","gravatar_id":"","url":"https://api.github.com/users/beikov","html_url":"https://github.com/beikov","followers_url":"https://api.github.com/users/beikov/followers","following_url":"https://api.github.com/users/beikov/following{/other_user}","gists_url":"https://api.github.com/users/beikov/gists{/gist_id}","starred_url":"https://api.github.com/users/beikov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/beikov/subscriptions","organizations_url":"https://api.github.com/users/beikov/orgs","repos_url":"https://api.github.com/users/beikov/repos","events_url":"https://api.github.com/users/beikov/events{/privacy}","received_events_url":"https://api.github.com/users/beikov/received_events","type":"User","site_admin":false},"assignees":[{"login":"beikov","id":358916,"avatar_url":"https://avatars3.githubusercontent.com/u/358916?v=3","gravatar_id":"","url":"https://api.github.com/users/beikov","html_url":"https://github.com/beikov","followers_url":"https://api.github.com/users/beikov/followers","following_url":"https://api.github.com/users/beikov/following{/other_user}","gists_url":"https://api.github.com/users/beikov/gists{/gist_id}","starred_url":"https://api.github.com/users/beikov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/beikov/subscriptions","organizations_url":"https://api.github.com/users/beikov/orgs","repos_url":"https://api.github.com/users/beikov/repos","events_url":"https://api.github.com/users/beikov/events{/privacy}","received_events_url":"https://api.github.com/users/beikov/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/Blazebit/blaze-persistence/milestones/7","html_url":"https://github.com/Blazebit/blaze-persistence/milestone/7","labels_url":"https://api.github.com/repos/Blazebit/blaze-persistence/milestones/7/labels","id":1151157,"number":7,"title":"1.2.0","description":"This release will be about\r\n\r\n* TREAT support\r\n* Table function support\r\n* Interval arithmetics support\r\n* Merge operation support\r\n* Entity view fetch strategies\r\n* Entity view partial updates (experimental support)\r\n* Entity view collection attribute filters","creator":{"login":"beikov","id":358916,"avatar_url":"https://avatars3.githubusercontent.com/u/358916?v=3","gravatar_id":"","url":"https://api.github.com/users/beikov","html_url":"https://github.com/beikov","followers_url":"https://api.github.com/users/beikov/followers","following_url":"https://api.github.com/users/beikov/following{/other_user}","gists_url":"https://api.github.com/users/beikov/gists{/gist_id}","starred_url":"https://api.github.com/users/beikov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/beikov/subscriptions","organizations_url":"https://api.github.com/users/beikov/orgs","repos_url":"https://api.github.com/users/beikov/repos","events_url":"https://api.github.com/users/beikov/events{/privacy}","received_events_url":"https://api.github.com/users/beikov/received_events","type":"User","site_admin":false},"open_issues":31,"closed_issues":98,"state":"open","created_at":"2015-06-05T13:35:11Z","updated_at":"2017-03-28T09:01:56Z","due_on":null,"closed_at":null},"comments":0,"created_at":"2017-02-23T18:55:30Z","updated_at":"2017-03-28T09:01:56Z","closed_at":"2017-03-28T09:01:56Z","body":"Right now, we always require an `@IdMapping` annotation within an entity view. Check if we can relax this requirement somehow and improve on some things\r\n* Do not require a value to be provided (we can infer the mapping)\r\n* Do not require the `@IdMapping` annotation explicitly if there is an attribute with the implicit mapping to the entities id attribute\r\n\r\nWe should also play around and see if it makes sense to have entity views that don't have a `@IdMapping`.\r\nThe only restrictions such an entity view would have are\r\n* ~~Can't have collections because elements can't be correlated without ids~~\r\n* ~~Can't be used in a collection mapping~~\r\n* ~~Can't be updatable because they don't have an id~~\r\n* ~~Can't be used with `PaginatedCriteriaBuilder` API because that requires an id~~\r\n\r\nThe only real limitation is that it can't have join fetched collections when no identity is available.\r\n\r\nUpdate the getting started guide when removing the need for `@IdMapping`\r\n\r\nKind of related to https://github.com/Blazebit/blaze-persistence/issues/376"}}
{"actor":{"display_login":"beikov"}}