Skip to content

Revert "Add validation for tiered storage in Upsert/Dedup tables"#17726

Merged
swaminathanmanish merged 1 commit intoapache:masterfrom
shounakmk219:revert-tier-check-dedup
Feb 19, 2026
Merged

Revert "Add validation for tiered storage in Upsert/Dedup tables"#17726
swaminathanmanish merged 1 commit intoapache:masterfrom
shounakmk219:revert-tier-check-dedup

Conversation

@shounakmk219
Copy link
Collaborator

Description

Reverting the changes made in #17064 as it blocks the dedup enabled tables from using multi tiers, for which support was added in #17154

@shounakmk219 shounakmk219 added bugfix dedup changes related to realtime ingestion dedup handling labels Feb 19, 2026
Copy link
Contributor

@swaminathanmanish swaminathanmanish left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov-commenter
Copy link

codecov-commenter commented Feb 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.19%. Comparing base (525a8d0) to head (75d3b09).

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #17726      +/-   ##
============================================
+ Coverage     63.16%   63.19%   +0.02%     
- Complexity     1499     1502       +3     
============================================
  Files          3181     3181              
  Lines        190816   190815       -1     
  Branches      29164    29163       -1     
============================================
+ Hits         120533   120578      +45     
+ Misses        60890    60851      -39     
+ Partials       9393     9386       -7     
Flag Coverage Δ
custom-integration1 100.00% <ø> (ø)
integration 100.00% <ø> (ø)
integration1 100.00% <ø> (ø)
integration2 0.00% <ø> (ø)
java-11 63.17% <ø> (+0.01%) ⬆️
java-21 63.16% <ø> (+29.12%) ⬆️
temurin 63.19% <ø> (+0.02%) ⬆️
unittests 63.18% <ø> (+0.02%) ⬆️
unittests1 55.59% <ø> (+0.04%) ⬆️
unittests2 34.03% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@swaminathanmanish swaminathanmanish merged commit 402e88a into apache:master Feb 19, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix dedup changes related to realtime ingestion dedup handling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments