On 3 Jan 2014, at 15:28, Sanne Grinovero <sa...@hibernate.org> wrote:
>> I think we dropped the ball on this one. I basically had a look at >> Guillaume’s pull request. >> His analysis was correct and his proposed patch brings back the old pre 4.4 >> behaviour >> with minimal changes. > > Ok that sounds good, I couldn't look at the PR yet.. should I not trust you ? > :) You definitely, should ;-) I just felt that we kind of left the discussion open ended. >> There is still the question of the indented use cases of @ContainedIn. As >> discussed in this thread >> afaik the indent was as a sole counterpart to @IndexedEmbedded. The >> implementation (probably >> as a side effect) made it also work when @IndexedEmbedded was not used. This >> was changed with >> the metamodel refactoring where contained in was indeed treated as >> counterpart of indexed embedded. >> >> I think we should go ahead and apply Guillaume’s pull request for 4.4 and >> 4.5, basically reinstating >> old 4.x behaviour, side effect or not. > > +1 Ok then >> The next question is then what to do in Search 5. I think we can just carry >> forward the change/patch. > > Yes let's keep them in sync for now > >> I don’t think that @ContainedIn needs another attribute or that we should >> introduce a whole new annotation. >> @ContainedIn seems quite natural provided we documented it intend and >> behaviour. Basically all what @ContainedIn >> is saying, is that here we have a reference to another entity which needs >> reindexing as well when the current >> entity gets reindexed. Whether the inclusions happens via @IndexEmbedded or >> via a custom bridge is irrelevant. >> >> Thoughts? > > I agree with you, but in insight if this new "meaning" would have been > explicit on this annotation from the beginning of time, maybe it would > have had a different name? Maybe. But what? @ContainedIn kind of fits. > I don't think the name is appropriate for this more extended meaning; > probably not a big problem but we can decide on that after it's fixed. Sure. > We don't have dates defined, but we can certainly release these when > there are enough good reasons to: for example if Guillaume needs it. ok _______________________________________________ hibernate-dev mailing list hibernate-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/hibernate-dev