[
https://issues.apache.org/jira/browse/HIVE-8292?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14167363#comment-14167363
]
Vikram Dixit K commented on HIVE-8292:
--
LGTM +1. +1 for 0.14 as well.
> Reading from
[
https://issues.apache.org/jira/browse/HIVE-8292?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14166941#comment-14166941
]
Hive QA commented on HIVE-8292:
---
{color:green}Overall{color}: +1 all checks pass
Here are
[
https://issues.apache.org/jira/browse/HIVE-8292?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14164223#comment-14164223
]
Gopal V commented on HIVE-8292:
---
The approach is similar to the other patch, but the IOContex
[
https://issues.apache.org/jira/browse/HIVE-8292?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14160729#comment-14160729
]
Gopal V commented on HIVE-8292:
---
[~mmokhtar]: Probably better to just read exec context off
[
https://issues.apache.org/jira/browse/HIVE-8292?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14160683#comment-14160683
]
Mostafa Mokhtar commented on HIVE-8292:
---
[~vikram.dixit]
Patch which addresses the re
[
https://issues.apache.org/jira/browse/HIVE-8292?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14159815#comment-14159815
]
Mostafa Mokhtar commented on HIVE-8292:
---
Looks like a regression introduced by
https
[
https://issues.apache.org/jira/browse/HIVE-8292?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14159293#comment-14159293
]
Gopal V commented on HIVE-8292:
---
Worse, I printed out the identity hashcode of the ExecMapper
[
https://issues.apache.org/jira/browse/HIVE-8292?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14158966#comment-14158966
]
Gopal V commented on HIVE-8292:
---
Traced it down to
{code}
@Override
public boolean pushR