Skip to content

docs: document null-handling function arguments#1527

Open
BharatDeva wants to merge 1 commit intoapache:mainfrom
BharatDeva:docs/function-null-argument-docs
Open

docs: document null-handling function arguments#1527
BharatDeva wants to merge 1 commit intoapache:mainfrom
BharatDeva:docs/function-null-argument-docs

Conversation

@BharatDeva
Copy link
Copy Markdown

Which issue does this PR close?

Closes #1463.

Rationale for this change

Some function docstrings in python/datafusion/functions.py include examples but do not describe their arguments. This makes the generated API docs harder to scan when users are looking up function signatures.

This PR makes a small, focused pass over the related null-handling helpers.

What changes are included in this PR?

Adds Args: sections for:

  • coalesce
  • ifnull
  • nanvl
  • nvl
  • nvl2

Are there any user-facing changes?

Yes, documentation-only. The generated API docs will include clearer argument descriptions for these functions. Runtime behavior is unchanged.

Validation

  • uv tool run ruff@0.15.1 check --config pyproject.toml python/datafusion/functions.py
    • Result: All checks passed!
  • uv tool run ruff@0.15.1 format --check --config pyproject.toml python/datafusion/functions.py
    • Result: 1 file already formatted
  • git diff --check
    • Result: passed with no whitespace errors

LLM-generated code disclosure

This documentation update was prepared with assistance from OpenAI Codex and manually reviewed before submission.

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.

Doc improvements

1 participant