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

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

paul-rogers commented on pull request #1871: DRILL-7403: lidate batch checks, 
vector integretity in unit tests
URL: https://github.com/apache/drill/pull/1871
 
 
   Enhances the existing record batch checks to check all the various
   batch record counts, and to more fully validate all vector types.
   
   This code revealed that virtually all record batches have
   problems: they omit setting some record count or other, they
   introduce some form of vector corruption.
   
   Since we want things to work as we make fixes, this change enables
   the checks for only one record batch: the "new" scan. Others are
   to come as they are fixed.
 
----------------------------------------------------------------
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:
us...@infra.apache.org


> Validate batch checks, vector integretity in unit tests
> -------------------------------------------------------
>
>                 Key: DRILL-7403
>                 URL: https://issues.apache.org/jira/browse/DRILL-7403
>             Project: Apache Drill
>          Issue Type: Improvement
>            Reporter: Paul Rogers
>            Assignee: Paul Rogers
>            Priority: Minor
>
> Drill provides a {{BatchValidator}} that checks vectors. It is disabled by 
> default. This enhancement adds more checks, including checks for row counts 
> (of which there are surprisingly many.)
> Since most operators will fail if the check is enabled, this enhancement also 
> adds a table to keep track of which operators pass the checks (and for which 
> checks should be enabled) and those that still need work. This allows the 
> checks to exist in the code, and to be enabled incrementally as we fix the 
> various problems.



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

Reply via email to