Skip to content

1261 - #1300

Merged
m-kovalsky merged 1 commit into
microsoft:mainfrom
m-kovalsky:m-kovalsky/1261
Jul 28, 2026
Merged

1261#1300
m-kovalsky merged 1 commit into
microsoft:mainfrom
m-kovalsky:m-kovalsky/1261

Conversation

@m-kovalsky

Copy link
Copy Markdown
Collaborator

Copilot AI review requested due to automatic review settings July 28, 2026 06:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Addresses Issue #1261 by preventing Delta Lake writes from failing when PyArrow infers an all-null column as the unsupported null type during VertiPaq Analyzer “export to table”.

Changes:

  • Added a resolve_null_columns() step to cast all-null PyArrow columns to a supported type (schema-specified type when available, otherwise string).
  • In pure-Python notebooks, converts pandas DataFrames to pyarrow.Table prior to write_deltalake, and applies the null-column resolution before writing.

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

Comment on lines +1053 to +1057
data = spark_df
if isinstance(data, pd.DataFrame):
data = pa.Table.from_pandas(data, preserve_index=False)
if isinstance(data, pa.Table):
data = resolve_null_columns(data)
@m-kovalsky
m-kovalsky merged commit e73861b into microsoft:main Jul 28, 2026
4 checks passed
@m-kovalsky
m-kovalsky deleted the m-kovalsky/1261 branch July 28, 2026 07:10
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