Skip to content

HIVE-26697: Create vectorized Hive DeleteFilter for Iceberg tables#3747

Merged
szlta merged 2 commits intoapache:masterfrom
szlta:HIVE-26697
Nov 11, 2022
Merged

HIVE-26697: Create vectorized Hive DeleteFilter for Iceberg tables#3747
szlta merged 2 commits intoapache:masterfrom
szlta:HIVE-26697

Conversation

@szlta
Copy link
Contributor

@szlta szlta commented Nov 10, 2022

Implement a DeleteFilter that changes an Iterable of VectorizedRowBatches in a way that whenever a new batch is seen in the next() method, we iterate its rows and apply the original Iceberg DeleteFilter on it. In VRBs we can utilize the selected array to mark rows as selected (not deleted), so essentially we're going to implement the markRowDeleted method.

Change-Id: I7b81bdbedcf908ddb6b17e3de27b54183ca085b3
Copy link
Contributor

@lcspinter lcspinter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

final +1 (pendig tests)

Change-Id: I1250527449a7f3e8c4f37ad80e38d9e635ea1452
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 9 Code Smells

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

@szlta szlta merged commit 8b48ce5 into apache:master Nov 11, 2022
dengzhhu653 pushed a commit to dengzhhu653/hive that referenced this pull request Dec 15, 2022
DongWei-4 pushed a commit to DongWei-4/hive that referenced this pull request Dec 29, 2022
…pache#3747) (Adam Szita, reviewed by Laszlo Pinter)

(cherry picked from commit 8b48ce5)
yeahyung pushed a commit to yeahyung/hive that referenced this pull request Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants