[
https://issues.apache.org/jira/browse/OAK-7151?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16327212#comment-16327212
]
Thomas Mueller commented on OAK-7151:
-------------------------------------
[~catholicon] The syntax "//*[jcr:contains(., 'foo')]/(rep:excerpt(.) | @a |
jcr:content/@b)" is to specify multiple column (excerpt, property a, property
jcr:content/b). This is already implemented. I see the syntax
"rep:excerpt(@foo1,@foo2,rel/@foo3)" would be much shorter than my version,
however it looks like it would be one column, while you actually want 3
columns. "rep:excerpt(@foo1) | rep:excerpt(@foo2) | rep:excerpt(rel/@foo3)" is
more clearly 3 columns. That's why I would prefer that syntax. I mean, in SQL,
you have to write "select lower(firstName), lower(lastName), lower(fullName)
from ...", and nobody recommended the alternative syntax "select
lower(firstName, lastName, fullName) from ...", even thought that syntax would
be shorter - it just looks like "lower" would be a function with 3 parameters,
which it is not.
> Support indexed based excerpts on properties
> --------------------------------------------
>
> Key: OAK-7151
> URL: https://issues.apache.org/jira/browse/OAK-7151
> Project: Jackrabbit Oak
> Issue Type: Improvement
> Components: lucene
> Reporter: Vikas Saurabh
> Assignee: Vikas Saurabh
> Priority: Major
> Fix For: 1.10
>
> Attachments: OAK-7151.patch, OAK-7151.xpath.patch
>
>
> As discovered in OAK-4401 we fallback to {{SimpleExcerptProvider}} when
> requesting excerpts for properties.
> The issue as highlighted in [~teofili]'s comment \[0] is that we at time of
> query we don't have information about which all columns/fields would be
> required for excerpts.
> A possible approach is that the query specified explicitly which columns
> would be required in facets (of course, node level excerpt would still be
> supported). This issue is to track that improvement.
> Note: this is *not* a substitute for OAK-4401 which is about doing saner
> highlighting when {{SimpleExcerptProvider}} comes into play e.g. despite this
> issue excerpt for non-stored fields (properties which aren't configured with
> {{useInExcerpt}} in the index definition}, we'd need to fallback to
> {{SimpleExcerptProvider}}.
> /[~tmueller]
> \[0]:
> https://issues.apache.org/jira/browse/OAK-4401?focusedCommentId=15299857&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15299857
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
