qphien commented on pull request #2171: URL: https://github.com/apache/iceberg/pull/2171#issuecomment-769705387
@pvary Thanks for your reply. I agree that creating an extra record is not a good way to solve vectorization problem in hive, we need to find another way to solve this problem. [TEZ-4248](https://issues.apache.org/jira/browse/TEZ-4248) is a better way to solve projection issue on tez. BTW, do we need to change [this assert](https://github.com/apache/iceberg/blob/19622dcfcb426485748fa017a6181e23df5732dc/mr/src/main/java/org/apache/iceberg/mr/hive/HiveIcebergSerDe.java#L112-L118) currently since this issue happens no matter whether `hive.vectorized.execution.enabled` is enabled ? ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
