[GitHub] [iceberg] liubo1022126 commented on pull request #2614: MR: Fix selectedColumns not belong to current Map (#2567)

[GitBox <gi...@apache.org>]

liubo1022126 commented on pull request #2614:
URL: https://github.com/apache/iceberg/pull/2614#issuecomment-844832360


   > @liubo1022126: Thanks for looking into this!
   > 
   > * Could you please create the PR against the master branch, and then we can port it to the 0.11 branch if needed (depending on the releases)
   > * Could you please add a test case which is failing before the fix and working after the fix? Maybe a new method into the `TestHiveIcebergStorageHandlerWithEngine`?
   > 
   > I am not entirely sure that I understand the root cause of the problem.
   > I feel that somehow the list of the projected columns are not correct. If my understanding is correct then this might be similar to #2171 but that is only for Tez execution engine.
   > Having a test case would greatly simplify the understanding of the issue.
   > 
   > Thanks,
   > Peter
   
   


-- 
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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org