You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@drill.apache.org by "Paul Rogers (Jira)" <ji...@apache.org> on 2019/10/27 18:52:00 UTC

[jira] [Created] (DRILL-7425) Remove redundant record count field from operators

Paul Rogers created DRILL-7425:
----------------------------------

             Summary: Remove redundant record count field from operators
                 Key: DRILL-7425
                 URL: https://issues.apache.org/jira/browse/DRILL-7425
             Project: Apache Drill
          Issue Type: Improvement
            Reporter: Paul Rogers
            Assignee: Paul Rogers


Work on the container count operator bugs revealed that multiple operators (Project, Unnest, probably others) maintain a record count field. As it turns out, the only semantically valid value for this field is to have the same value as the container record count. Hence, the record count field is redundant, and is just another detail to get right.

The only real use of the variable is to report a record count of 0 before the first batch is created. Because of the way the container reports counts, it will throw an exception after the container is created, before the value is set to 0.

The goal of this ticket is to remove the variable, addressing any resulting issues.



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