steigma commented on PR #2504:
URL: https://github.com/apache/tinkerpop/pull/2504#issuecomment-1966731520
Thanks for the feedback. I added a few tests and renamed some variables. I
also found that the lambda handling was not really working since the step
collection was restricted to TraversalParent subclasses. This should now also
be fixed and I added some basic test coverage for this.
In our experiments with queries of the form
```
g.V('vertex')...
coalesce(
and(
values('property').is('val1'),
values('property').is('val2'),
...
values('property').is('val800')
),
sideEffect(properties('property').drop()).
property(set, 'propery', 'val1').
property(set, 'propery', 'val2').
...
property(set, 'propery', 'val800')
)...
```
we have seen improvement of around 25% including all strategies and
executing the queries, i.e,. the FilterRankingStrategy improved significantly
(as also other strategies taking some significant amount of time).
--
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]
