On 29 Apr 2014, at 15:09, Sanne Grinovero <sa...@hibernate.org> wrote:

> I'd actually like to propose to change the depth default to zero, and
> since includePath also defaults to an empty list, we'd be logging a
> warning as the @IndexedEmbedded annotation would have no effect.
> 
> 
>> I also find it not intuitive that this means:
>> "index all embedded fields up to the given depth, plus the specified paths”.
>> I never liked how we baked depth and includePath into the same annotation. A 
>> dedicated annotation would
>> have been more appropriate.
> 
> We've been there and couldn't agree on a better proposal. Feel free to
> reopen the case on a new thread, if you have a nice name in mind. But
> ultimately remember the goal is to allow queries on a well-known list
> of field names, and it would be great to validate for these queries,
> so to have a clear definitions of which indexing and analysis options
> are applied to each Lucene field, and how to apply a bi-directional
> projection.. so I'm still skeptical on leaving too much freedoom, or
> have multiple ways to achieve the same thing.

Right, splitting the annotation in two does not address the mix of both 
concepts.
_______________________________________________
hibernate-dev mailing list
hibernate-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/hibernate-dev

Reply via email to