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

Posted to issues@hive.apache.org by "Seonggon Namgung (Jira)" <ji...@apache.org> on 2025/08/25 08:21:00 UTC

[jira] [Commented] (HIVE-29160) Replace forward() in vectorized operators with vectorForward().

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

Seonggon Namgung commented on HIVE-29160:
-----------------------------------------

Merged to master.

Thanks to [~dkuzmenko], [~ayushtkn], and [~InvisibleProgrammer] for the review!

> Replace forward() in vectorized operators with vectorForward().
> ---------------------------------------------------------------
>
>                 Key: HIVE-29160
>                 URL: https://issues.apache.org/jira/browse/HIVE-29160
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Seonggon Namgung
>            Assignee: Seonggon Namgung
>            Priority: Minor
>              Labels: pull-request-available
>             Fix For: 4.2.0
>
>
> I found that VectorAMEventOperator and VectorPTFOperator use the raw forward() instead of vectorForward(). Since using forward() in a vectorized operator leads to an incorrect count of processed rows and may cause wrong results when there are multiple child operators, these calls should be updated to use the proper version.



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