Skip to content

docs: add multiple conditional clauses examples to merge docs - #4890

Merged
ion-elgreco merged 2 commits into
delta-io:mainfrom
Shahzeb-DataEngineer:docs/merge-conditional-clauses
Oct 20, 2026
Merged

ion-elgreco merged 2 commits into
delta-io:mainfrom
Shahzeb-DataEngineer:docs/merge-conditional-clauses

Conversation

@Shahzeb-DataEngineer

Copy link
Copy Markdown
Contributor

Description

The merge docs showed per-clause predicates only for delete, and did not show how several clauses of the same kind combine. This adds a "Multiple conditional clauses" section to docs/usage/merging-tables.md demonstrating:

  • Two when_matched_update clauses with different per-clause predicates (first match wins): flag orders with a >50% amount jump for manual review, apply the new amount directly otherwise.
  • A conditional when_not_matched_insert: only insert source rows satisfying an extra predicate.

Both examples were verified against deltalake 1.6.6 with the exact outputs shown in the docs.

Related Issue

Closes #3345

Copilot AI balanced review requested due to automatic review settings October 16, 2026 04:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Oct 16, 2026
@codecov

codecov Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.94%. Comparing base (a32a678) to head (8966289).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4890      +/-   ##
==========================================
- Coverage   81.95%   81.94%   -0.01%     
==========================================
  Files         200      200              
  Lines       75769    75769              
  Branches    75769    75769              
==========================================
- Hits        62094    62092       -2     
- Misses      10042    10045       +3     
+ Partials     3633     3632       -1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

The merge docs showed per-clause predicates only for delete and
did not show how several clauses of the same kind combine.
Add a section demonstrating two when_matched_update clauses with
different predicates (first match wins) and a conditional
when_not_matched_insert. Both examples were verified against
deltalake 1.6.6.

Closes delta-io#3345

Signed-off-by: Shahzeb Ali <[email protected]>
@ion-elgreco
ion-elgreco force-pushed the docs/merge-conditional-clauses branch from 3778596 to 5849c75 Compare October 20, 2026 11:53
@ion-elgreco
ion-elgreco enabled auto-merge (squash) October 20, 2026 11:53
@ion-elgreco
ion-elgreco merged commit 7e686e6 into delta-io:main Oct 20, 2026
34 of 35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Documentation improvement request: Merge table

3 participants