Skip to content

Work on solving #2410: first add a failing IT#2411

Merged
tatu-at-datastax merged 4 commits intomainfrom
tatu/2410-insert-one-table-udt-with-inet
Feb 26, 2026
Merged

Work on solving #2410: first add a failing IT#2411
tatu-at-datastax merged 4 commits intomainfrom
tatu/2410-insert-one-table-udt-with-inet

Conversation

@tatu-at-datastax
Copy link
Contributor

What this PR does:

Will fix #2410: insertOne for table with UDT with inet column fails

Which issue(s) this PR fixes:
Fixes #2410

Checklist

  • Changes manually tested
  • Automated Tests added/updated
  • Documentation added/updated
  • CLA Signed: DataStax CLA

@github-actions
Copy link
Contributor

github-actions bot commented Feb 26, 2026

📉 Unit Test Coverage Delta vs Main Branch

Metric Value
Main Branch 50.07%
This PR 50.07%
Delta 🔴 -0.00%
⚠️ Coverage decreased

@github-actions
Copy link
Contributor

github-actions bot commented Feb 26, 2026

Unit Test Coverage Report

Overall Project 50.07% -0.01% 🍏
Files changed 0%

File Coverage
UDTCodecs.java 0% -3.51%

@github-actions
Copy link
Contributor

📈 Integration Test Coverage Delta vs Main Branch (dse69-it)

Metric Value
Main Branch 70.39%
This PR 70.39%
Delta 🟢 +0.00%
✅ Coverage improved!

@github-actions
Copy link
Contributor

Integration Test Coverage Report (dse69-it)

Overall Project 70.39% 🍏
Files changed 100% 🍏

File Coverage
UDTCodecs.java 76.9% 🍏

@github-actions
Copy link
Contributor

📈 Integration Test Coverage Delta vs Main Branch (hcd-it)

Metric Value
Main Branch 72.13%
This PR 72.13%
Delta 🟢 +0.00%
✅ Coverage improved!

@github-actions
Copy link
Contributor

Integration Test Coverage Report (hcd-it)

Overall Project 72.13% 🍏
Files changed 100% 🍏

File Coverage
UDTCodecs.java 76.9% 🍏

@tatu-at-datastax tatu-at-datastax changed the title (WIP) Work on solving #2410: first add a failing IT Work on solving #2410: first add a failing IT Feb 26, 2026
@tatu-at-datastax tatu-at-datastax marked this pull request as ready for review February 26, 2026 19:35
@tatu-at-datastax tatu-at-datastax requested a review from a team as a code owner February 26, 2026 19:35
Copy link
Contributor

@sl-at-ibm sl-at-ibm left a comment

Choose a reason for hiding this comment

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

LGTM (I ran the full battery of UTD tests of astrapy, original and with added UDT fields, and everything was shining green).

@tatu-at-datastax tatu-at-datastax merged commit d600d3f into main Feb 26, 2026
3 checks passed
@tatu-at-datastax tatu-at-datastax deleted the tatu/2410-insert-one-table-udt-with-inet branch February 26, 2026 23:25
@tatu-at-datastax
Copy link
Contributor Author

Thank you @sl-at-ibm !

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.

Bug: No codec found for inet type within UDT for the write path

2 participants