[jira] [Commented] (DRILL-7333) Batch of container count fixes

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

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

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

paul-rogers commented on pull request #1830: DRILL-7333: Set container record count where missing
URL: https://github.com/apache/drill/pull/1830
 
 
   Many operators fail to set the container record count. See DRILL-7333
   for details. This commit fixes a large number of operators. Replaces
   blocks of copy/paste code to set the container vector value count with
   a call to a single function that does the work. That function also
   sets the count for the container as a whole, something that many
   operators failed to do, causing hard-to-find errors downstream.
   
   Tests: reran all unit tests. A new unit test, to verify that the
   count is set, will come in a later commit once all fixes are made.
   
   Also contains code cleanup for items noticed while finding the container count issues.
 
----------------------------------------------------------------
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


> Batch of container count fixes
> ------------------------------
>
>                 Key: DRILL-7333
>                 URL: https://issues.apache.org/jira/browse/DRILL-7333
>             Project: Apache Drill
>          Issue Type: Bug
>    Affects Versions: 1.16.0
>            Reporter: Paul Rogers
>            Assignee: Paul Rogers
>            Priority: Major
>             Fix For: 1.17.0
>
>
> See DRILL-7325. The set of operators to be fixed is large. This ticket covers a subset of operators.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)