[
https://issues.apache.org/jira/browse/IGNITE-12291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17346175#comment-17346175
]
Andrey Mashenkov commented on IGNITE-12291:
-------------------------------------------
[~timonin.maksim], you wrote:
{noformat}
So, I think, that number of pages to load can be limited with different
approach that described in the ticket description. We can move check of limit
to the `next()` call instead of `enqueue()`. After `next()` returns NULL
iterator automatically closes and send cancel requests to all nodes. {noformat}
You saves data into collection that will be never used.
What issue does the logic moving fix?
Lucene returns result sorted by relevance/score, but we might miss this
information somewhere.
> Create controllable paged query requests / responses for TextQuery similar to
> current SQL result processing
> -----------------------------------------------------------------------------------------------------------
>
> Key: IGNITE-12291
> URL: https://issues.apache.org/jira/browse/IGNITE-12291
> Project: Ignite
> Issue Type: Improvement
> Reporter: Yuriy Shuliha
> Assignee: Maksim Timonin
> Priority: Major
> Time Spent: 10m
> Remaining Estimate: 0h
>
> For now query initiator node sends 1 _GridCacheQueryRequest_ and can get
> multiple _GridCacheQueryResponse_-s per remote.
> TextQuery processing finishes when all remotes send _GridCacheQueryResponse_
> with 'finished' flag.
> _TextQuery_ processingĀ should be reworked like it was done for SQL queries.
> Ignite has _GridQueryNextPageRequest \ Response_ classes for SQL result
> processing.
> Similar processing should be implemented for _TextQueries_:
> *GridTextQueryNextPageRequest*
> *GridTextQueryNextPageResponse*
> Proper _TextQuery_ response processing should be implemented inĀ
> _GridCacheQueryFutureAdapter._
> This will allow us to add correct sorting and apply limit correctly on reduce.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)