[ 
https://issues.apache.org/jira/browse/SOLR-16556?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17683929#comment-17683929
 ] 

Eric Pugh commented on SOLR-16556:
----------------------------------

[~mnrathod] thanks for pinging me.   

I managed to get [~gabrielmagno] to look at your patch with me, to give another 
perspective.  I don't have [~jbernste] deep understanding of the core 
principles under pinning the streaming expressions, so I'm a bit out of my 
depth.

Here are some thoughts on maybe how to move forward with this contribution to 
get it committed.

1) [~gabrielmagno] pointed out that this really feels like a "slice" operator 
versus a "paging" operator.   In my mind, the word "paging" is really about the 
concept that I'll iteratively work through the data set by updating my start 
parameter over time and is very search world specific.   What I see is 
something that can create a sub set of tuples from any source...   Could be a 
search, but it could also be from a CSV file!   What do you think about 
renaming this to "slice" instead?

2) I like the simplicity of the operators over the interesting approach Joel 
provided.  It's cool we can use the recnum() feature, but it's definitely more 
complex approach!   

3) Have you done any performance testing of combining this operator, call it 
"slice", with a "sort" operator, which might simplify this code base?   We 
looked a little bit to try and understand if you did a slice, and wrapped it 
with a sort, would you get the same thing?  Just trying to think about if this 
would simplify the logic conceptually a bit?   Something like 
sort(slice(expression, start, rows), by="fields").





> Solr stream expression: Implement Page Streaming Decorator to allow results 
> to be displayed with pagination.
> ------------------------------------------------------------------------------------------------------------
>
>                 Key: SOLR-16556
>                 URL: https://issues.apache.org/jira/browse/SOLR-16556
>             Project: Solr
>          Issue Type: Improvement
>          Components: streaming expressions
>            Reporter: Maulin
>            Priority: Major
>              Labels: Streamingexpression, decorator, paging
>         Attachments: Page Decorator Performance Reading.xlsx
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> Solr stream expression: Implement Page Streaming Decorator to allow results 
> to be displayed with pagination.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org

Reply via email to