Skip to content

chore: Update spark_expressions_support.md doc#4165

Merged
andygrove merged 3 commits intoapache:mainfrom
comphead:doc
May 1, 2026
Merged

chore: Update spark_expressions_support.md doc#4165
andygrove merged 3 commits intoapache:mainfrom
comphead:doc

Conversation

@comphead
Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Closes #.

Rationale for this change

What changes are included in this PR?

How are these changes tested?

@comphead comphead requested a review from andygrove April 30, 2026 19:56
@comphead comphead changed the title Update spark_expressions_support.md doc chore: Update spark_expressions_support.md doc Apr 30, 2026
Comment on lines +27 to +29
a sub-bullet labeled **Known issues** links to the relevant category page of the
[Compatibility Guide](../user-guide/latest/compatibility/expressions/index.md), where per-function
details are documented.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

With this, we would now have to maintain this information manually and keep it in sync with the compatibility guide. For example, if we fix a compatibility issue we have to update this doc as well.

Can we just make this more generic and say "all known compatibility issues are documented in the compatibility guide"?

Comment on lines -92 to -94
- Spark 3.4.3 audited 2026-04-02
- Spark 3.5.8 audited 2026-04-02
- Spark 4.0.1 audited 2026-04-02 (pos=0 error message differs from Spark)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

we don't want to delete the audit log

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

reverting

Copy link
Copy Markdown
Member

@andygrove andygrove left a comment

Choose a reason for hiding this comment

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

LGTM

@andygrove andygrove merged commit 050320d into apache:main May 1, 2026
5 checks passed
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.

2 participants