nit0906 commented on code in PR #853:
URL: https://github.com/apache/jackrabbit-oak/pull/853#discussion_r1115225573
##########
oak-search-elastic/src/main/java/org/apache/jackrabbit/oak/plugins/index/elastic/query/ElasticIndexPlanner.java:
##########
@@ -48,6 +53,8 @@ protected List<QueryIndex.OrderEntry>
createSortOrder(IndexDefinition.IndexingRu
PropertyDefinition pd = rule.getConfig(propName);
if (pd != null
&& o.getPropertyType() != null
+ // functions on regexp-based properties must be skipped
since the values cannot be indexed
+ && (!pd.isRegexp ||
!propName.startsWith(QueryConstants.FUNCTION_RESTRICTION_PREFIX))
Review Comment:
Do we want to ignore those properties that have regex enabled and have a
function defined or either of these ? As per the comment it should be the first
one but the condition is for the latter.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
