You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "itholic (via GitHub)" <gi...@apache.org> on 2023/11/24 02:12:54 UTC
[PR] [MINOR] Update Description of `UnknownException` in PySpark Documentation [spark]
itholic opened a new pull request, #43994:
URL: https://github.com/apache/spark/pull/43994
### What changes were proposed in this pull request?
This PR proposes to update the description of the `UnknownException` class in the PySpark documentation. The current description, "None of the above exceptions.", is being changed to "None of the other exceptions."
### Why are the changes needed?
The existing description, "None of the above exceptions.", implies that `UnknownException` does not cover any exceptions listed after it in the documentation. This can be misleading as there may be other exceptions defined below `UnknownException` as below:
<img width="757" alt="Screenshot 2023-11-24 at 11 11 26 AM" src="https://github.com/apache/spark/assets/44108233/93483c7e-1be0-41b1-8a74-7a667e8c603c">
The proposed change to "None of the other exceptions." provides a clearer and more accurate description, ensuring that it encompasses all exceptions not covered by previously listed specific exceptions.
### Does this PR introduce _any_ user-facing change?
No, it only involves a minor update to the documentation for clarity.
### How was this patch tested?
The existing CI should pass, especially PySpark documentation build.
### Was this patch authored or co-authored using generative AI tooling?
No.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
Re: [PR] [MINOR][DOCS] Update Description of `UnknownException` in PySpark Documentation [spark]
Posted by "HyukjinKwon (via GitHub)" <gi...@apache.org>.
HyukjinKwon closed pull request #43994: [MINOR][DOCS] Update Description of `UnknownException` in PySpark Documentation
URL: https://github.com/apache/spark/pull/43994
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
Re: [PR] [MINOR][DOCS] Update Description of `UnknownException` in PySpark Documentation [spark]
Posted by "HyukjinKwon (via GitHub)" <gi...@apache.org>.
HyukjinKwon commented on PR #43994:
URL: https://github.com/apache/spark/pull/43994#issuecomment-1825068655
Merged to master.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org