Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HIVE-26694: Populate file row position information during vectorized Iceberg reads #3732

Merged
merged 1 commit into from Nov 7, 2022

Conversation

szlta
Copy link
Contributor

@szlta szlta commented Nov 4, 2022

Currently we leverage VectorizedOrcRecordReader and VectorizedParquetRecordReader during vectorized Iceberg reads. In order to read V2 tables where delete files with positional deletes are present, we need to make these record reader implementations provide the row number information too.

Change-Id: I2f66c0529a28eb581a0b58928969818ab0cc19cd
@szlta szlta requested a review from lcspinter November 4, 2022 14:26
@szlta szlta changed the title HIVE-26694 HIVE-26694: Populate file row position information during vectorized Iceberg reads Nov 4, 2022
@sonarcloud
Copy link

sonarcloud bot commented Nov 4, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug B 2 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 10 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@szlta szlta merged commit 63b6134 into apache:master Nov 7, 2022
dengzhhu653 pushed a commit to dengzhhu653/hive that referenced this pull request Dec 15, 2022
…Iceberg reads (apache#3732) (Adam Szita, reviewed by Laszlo Pinter)
DongWei-4 pushed a commit to DongWei-4/hive that referenced this pull request Dec 29, 2022
…Iceberg reads (apache#3732) (Adam Szita, reviewed by Laszlo Pinter)

(cherry picked from commit 63b6134)
yeahyung pushed a commit to yeahyung/hive that referenced this pull request Jul 20, 2023
…Iceberg reads (apache#3732) (Adam Szita, reviewed by Laszlo Pinter)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants