Skip to content

[DOP-29429] Add Iceberg integration tests#288

Merged
IlyasDevelopment merged 1 commit intodevelopfrom
misc/DOP-29429
Oct 30, 2025
Merged

[DOP-29429] Add Iceberg integration tests#288
IlyasDevelopment merged 1 commit intodevelopfrom
misc/DOP-29429

Conversation

@IlyasDevelopment
Copy link
Copy Markdown
Contributor

@IlyasDevelopment IlyasDevelopment commented Oct 30, 2025 •
edited
Loading

Change Summary

Added integration tests for basic read/write with Iceberg connection + REST catalog + S3 warehouse

Checklist

  • Commit message and PR title is comprehensive
  • Keep the change as small as possible
  • Unit and integration tests for the changes exist
  • Tests pass on CI and coverage does not decrease
  • Documentation reflects the changes where applicable
  • docs/changelog/next_release/<pull request or issue id>.<change type>.rst file added describing change
    (see CONTRIBUTING.rst for details.)
  • My PR is ready to review.

@IlyasDevelopment IlyasDevelopment self-assigned this Oct 30, 2025
@IlyasDevelopment IlyasDevelopment added the ci:skip-changelog Add this label to skip changelog file check label Oct 30, 2025
dolfinus
dolfinus reviewed Oct 30, 2025
View reviewed changes
Comment thread .github/workflows/iceberg-tests.yml Outdated
dolfinus
dolfinus reviewed Oct 30, 2025
View reviewed changes
Comment thread README.rst Outdated
dolfinus
dolfinus reviewed Oct 30, 2025
View reviewed changes
Comment thread syncmaster/worker/spark.py Outdated
@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 30, 2025 •
edited
Loading

Coverage

Coverage Report •
FileStmtsMissCoverMissing
syncmaster/worker/handlers/db
   iceberg.py46197%81
TOTAL503728094% 

dolfinus
dolfinus reviewed Oct 30, 2025
View reviewed changes
Comment thread syncmaster/worker/spark.py Outdated
@IlyasDevelopment IlyasDevelopment merged commit 7918c19 into develop Oct 30, 2025
24 of 25 checks passed
@IlyasDevelopment IlyasDevelopment deleted the misc/DOP-29429 branch October 30, 2025 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dolfinus dolfinus dolfinus approved these changes

@TiGrib TiGrib Awaiting requested review from TiGrib

@marashka marashka Awaiting requested review from marashka

Assignees

@IlyasDevelopment IlyasDevelopment

Labels

ci:skip-changelog Add this label to skip changelog file check

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@IlyasDevelopment @dolfinus