GH-50897: [Python] Fix typo LZ0 -> LZO in ORC writer docstring - #50898
Conversation
Signed-off-by: Maulik Patel <21225464+maupatel@users.noreply.github.com>
|
|
|
|
|
Could you use our PR template? |
Reranko05
left a comment
There was a problem hiding this comment.
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.
|
|
thisisnic
left a comment
There was a problem hiding this comment.
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!
|
|
There was a problem hiding this comment.
🟢 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→LZOin the ORC writercompressiondocstring.
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.
Rationale for this change
Fix a typo in the ORC writer
compressiondocstring whereLZ0was incorrectly written instead ofLZO.What changes are included in this PR?
Change
LZ0toLZOin thepython/pyarrow/orc.pydocumentation.Are these changes tested?
Documentation-only change; no code behavior is affected.
Are there any user-facing changes?
No