MINOR: Fix missing cast to str #18016

IceS2 · 2024-09-26T14:18:30Z

After migratint to Pydantic V2, we need to explicitely cast URLs to string.
This one was missing.

I have read the CONTRIBUTING document.
My PR title is Fixes <issue-number>: <short explanation>
I have commented on my code, particularly in hard-to-understand areas.
For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

sonarcloud · 2024-09-27T11:40:45Z

Quality Gate passed for 'open-metadata-ingestion'

Fix missing cast to str

1763f0d

IceS2 requested a review from a team as a code owner September 26, 2024 14:18

IceS2 temporarily deployed to test September 26, 2024 14:18 — with GitHub Actions Inactive

IceS2 had a problem deploying to test September 26, 2024 14:18 — with GitHub Actions Failure

github-actions bot added Ingestion safe to test Add this label to run secure Github workflows on PRs labels Sep 26, 2024

ulixius9 approved these changes Sep 26, 2024

View reviewed changes

Merge branch 'main' into fix-iceberg-rest-catalog-config

22df298

IceS2 temporarily deployed to test September 27, 2024 10:31 — with GitHub Actions Inactive

IceS2 had a problem deploying to test September 27, 2024 10:31 — with GitHub Actions Failure

IceS2 merged commit c9d37f6 into main Sep 27, 2024
14 of 16 checks passed

IceS2 deleted the fix-iceberg-rest-catalog-config branch September 27, 2024 14:03

ulixius9 pushed a commit that referenced this pull request Sep 30, 2024

Fix missing cast to str (#18016)

cb2ef3f

Prajwal214 mentioned this pull request Oct 1, 2024

Iceberg Connector Ingestion Issue with pydantic V2 #18012

Closed