[jira] [Commented] (DRILL-7734) Revise the result set reader

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

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

ASF GitHub Bot commented on DRILL-7734:
---------------------------------------

paul-rogers opened a new pull request #2077:
URL: https://github.com/apache/drill/pull/2077


   # [DRILL-7734](https://issues.apache.org/jira/browse/DRILL-7734): Revise the result set reader
   
   ## Description
   
   The "result set reader" uses the column accessors to iterate over rows from multiple batches, similar to how the "result set loader" creates batches in a scan. This PR refactors the code to clarify the two ways that the reader is used. First, the "pull" reader in one operator reads ("pulls") batches from an upstream operator. To implement JSON streaming, we need a second model, a "push" version where a caller provides batches.
   
   ## Documentation
   N/A
   
   ## Testing
   Added tests. Reran all unit tests.
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> Revise the result set reader
> ----------------------------
>
>                 Key: DRILL-7734
>                 URL: https://issues.apache.org/jira/browse/DRILL-7734
>             Project: Apache Drill
>          Issue Type: Improvement
>    Affects Versions: 1.17.0
>            Reporter: Paul Rogers
>            Assignee: Paul Rogers
>            Priority: Major
>             Fix For: 1.18.0
>
>
> Updates to the {{ResultSetReader}} abstractions to make them usable in more cases.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)