Skip to content

[MCC-1514065] Python SDK – Enhance Validation Failure Diagnostics: Expose Invalid Reason in dry_publish() and publish()#48

Merged
slingampalli-mdsol merged 2 commits into
mainfrom
fix/MCC-1514065_pub_invalid_records
Jun 12, 2026
Merged

[MCC-1514065] Python SDK – Enhance Validation Failure Diagnostics: Expose Invalid Reason in dry_publish() and publish()#48
slingampalli-mdsol merged 2 commits into
mainfrom
fix/MCC-1514065_pub_invalid_records

Conversation

@ibaig-mdsol

@ibaig-mdsol ibaig-mdsol commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

enhance

Checklist

  • Review the pull request to fix typos and ensure variable/function names are intuitive, etc.
  • Make sure the pull request does not have excessive number of unnecessary commits. Utilize the git commit --amend --no-edit command to reduce commit messages when making small file changes (like changing linespacing).
  • Make sure you have added unit tests for the code changes. Tests should be added in the tests/ folder.
  • Modify docs, if required. Add any new documentation in the doc/ folder.
  • Rebase on latest active development branch (develop/main).

Changes Summary

  • Handling invalid records in the usage file for publish and dry_publish()

Copilot AI review requested due to automatic review settings June 12, 2026 21:54

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the DataConnect Python SDK usage guide to improve diagnostics when dry_publish()/publish() encounter invalid records, by showing additional server-returned details (notably the invalid_records table, which can include invalid reasons).

Changes:

  • In the dry publish example, conditionally prints errors, invalid_datetime_formats, and the invalid_records DataFrame when present.
  • In the publish example, prints the invalid_records DataFrame when present.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread guides/dataconnect_usage.ipynb
@slingampalli-mdsol slingampalli-mdsol merged commit 4ef0f8e into main Jun 12, 2026
3 checks passed
@slingampalli-mdsol slingampalli-mdsol deleted the fix/MCC-1514065_pub_invalid_records branch June 12, 2026 22:30
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.

3 participants