[
https://issues.apache.org/jira/browse/SOLR-15156?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17341453#comment-17341453
]
ASF subversion and git services commented on SOLR-15156:
--------------------------------------------------------
Commit efac0edee50ddd9beba765a86a4b591027cfdc60 in lucene-solr's branch
refs/heads/branch_8x from David Smiley
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=efac0ed ]
SOLR-15156: [child childFilter='...:...'] no longer escapes (#2367)
The query escaping it did was inconsistent with all other places in Solr where
a Lucene query may be provided.
There was no escaping here prior to 8.0 either.
> child transformer's childFilter shouldn't do escaping
> -----------------------------------------------------
>
> Key: SOLR-15156
> URL: https://issues.apache.org/jira/browse/SOLR-15156
> Project: Solr
> Issue Type: Improvement
> Reporter: David Smiley
> Assignee: David Smiley
> Priority: Major
> Fix For: main (9.0)
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> The {{child}} doc transformer's {{childFilter}} supports both some special
> path syntax and a simple field:value query with built-in escaping, or some
> other query so long as it doesn't contain a colon. I think there should not
> have been escaping here in the first place, thus letting you express whatever
> interesting query you want like we do in most other places in Solr. Simple.
> It may mean the user has to escape characters like '/' but that's normal. In
> a Java String that has an embedded local-params for the child doc
> transformer, that winds up being four back-slashes. So be it. It may seem
> like we're trying to do the user a favor by escaping what we see but I think
> it's actually more confusing to reason about because it's inconsistent, and
> it's limiting by getting in the way of more interesting queries. Besides,
> there are other ways of constructing the query syntax that results in zero
> escaping like {{childFilter='\{!field f=_nest_path_}/toppings'}} So that's
> longer but no escaping at least. Up to the user's prerogative.
> This will break existing users who depend on the escaping, and so warrants
> being done in a major release.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org
