Skip to content

Conversation

@srawat98-dev
Copy link
Contributor

@srawat98-dev srawat98-dev commented Nov 20, 2025

Summary

Issue] Briefly discuss the summary of the changes made in this
pull request in 2-3 lines.
Iceberg repo version bump.
This bump brings in :

  1. Iceberg Pull - 213
  2. Iceberg Pull - 212
    This would be adding app-name in snapshots metadata table. We won't require to join app-id with rmmapsummary table.

Changes

  • Client-facing API Changes
  • Internal API Changes
  • Bug Fixes
  • New Features
  • Performance Improvements
  • Code Style
  • Refactoring
  • Documentation
  • Tests

For all the boxes checked, please include additional details of the changes made in this pull request.

Testing Done

  • Manually Tested on local docker setup. Please include commands ran, and their output.
  • Added new tests for the changes made.
  • Updated existing tests to reflect the changes made.
  • No tests added or updated. Please explain why. If unsure, please feel free to ask for help.
  • Some other form of testing like staging or soak time in production. Please explain.

For all the boxes checked, include a detailed description of the testing done for the changes made in this pull request.

Additional Information

  • Breaking Changes
  • Deprecations
  • Large PR broken into smaller PRs, and PR plan linked in the description.

For all the boxes checked, include additional details of the changes made in this pull request.

@srawat98-dev srawat98-dev marked this pull request as ready for review November 20, 2025 23:23
@cbb330 cbb330 merged commit c03fae3 into linkedin:main Nov 21, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants