Skip to content

GH-50897: [Python] Fix typo LZ0 -> LZO in ORC writer docstring - #50898

Merged
thisisnic merged 1 commit into
apache:mainfrom
maupatel:gh-50897-orc-lzo-typo
Sep 4, 2026
Merged

thisisnic merged 1 commit into
apache:mainfrom
maupatel:gh-50897-orc-lzo-typo

Conversation

@maupatel

@maupatel maupatel commented Aug 17, 2026 •

Copy link
Copy Markdown
Contributor

Rationale for this change

Fix a typo in the ORC writer compression docstring where LZ0 was incorrectly written instead of LZO.

What changes are included in this PR?

Change LZ0 to LZO in the python/pyarrow/orc.py documentation.

Are these changes tested?

Documentation-only change; no code behavior is affected.

Are there any user-facing changes?

No

Signed-off-by: Maulik Patel <21225464+maupatel@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown

⚠️ GitHub issue #50897 has been automatically assigned in GitHub to PR creator.

@github-actions

Copy link
Copy Markdown

⚠️ GitHub issue #50897 has no components, please add labels for components.

@kou

kou commented Aug 18, 2026

Copy link
Copy Markdown
Member

Could you use our PR template?

@kou
kou marked this pull request as draft August 18, 2026 00:28

@Reranko05 Reranko05 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please use this PR template for the description:

### Rationale for this change

Fix a typo in the ORC writer `compression` docstring where `LZ0` was incorrectly written instead of `LZO`.

### What changes are included in this PR?

Change `LZ0` to `LZO` in the `python/pyarrow/orc.py` documentation.

### Are these changes tested?

Documentation-only change; no code behavior is affected.

### Are there any user-facing changes?

Yes

- GitHub Issue: #50897

Once updated, you can mark the PR as ready for review.

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

⚠️ GitHub issue #50897 has no components, please add labels for components.

@github-actions github-actions Bot added awaiting committer review Awaiting committer review and removed awaiting review Awaiting review labels Sep 4, 2026

@thisisnic thisisnic left a comment

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.

For the sake of keeping things moving, I've made the update to the PR description myself. FWIW @maupatel in future, please feel free to make a PR for something like this with no associated issue and the "MINOR" prefix in the PR name. Thanks for spotting and updating!

@thisisnic
thisisnic marked this pull request as ready for review September 4, 2026 11:03
Copilot AI lite review requested due to automatic review settings September 4, 2026 11:03
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

⚠️ GitHub issue #50897 has no components, please add labels for components.

@github-actions github-actions Bot added awaiting merge Awaiting merge and removed awaiting committer review Awaiting committer review labels Sep 4, 2026

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.

🟢 Approval recommended

The change is a straightforward documentation typo fix that matches the stated issue and does not affect runtime behavior.

Pull request overview

Fixes a user-facing documentation typo in the PyArrow ORC writer compression parameter docstring by correcting the codec name from LZ0 (invalid) to LZO (correct ORC compression algorithm name), aligning the docstring with expected terminology.

Changes:

  • Correct LZ0 → LZO in the ORC writer compression docstring.
File summaries
File Description
python/pyarrow/orc.py Corrects a docstring typo for the ORC writer compression codec name (LZ0 → LZO).
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@thisisnic
thisisnic merged commit 8fabb87 into apache:main Sep 4, 2026
45 of 47 checks passed
@thisisnic thisisnic removed the awaiting merge Awaiting merge label Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants