You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "panbingkun (via GitHub)" <gi...@apache.org> on 2024/03/12 02:01:53 UTC
[PR] [MINOR][DOCS] Remove the extra text on page sql-error-conditions-sqlstates [spark]
panbingkun opened a new pull request, #45469:
URL: https://github.com/apache/spark/pull/45469
### What changes were proposed in this pull request?
The pr aims to remove the extra text `.. include:: /shared/replacements.md` on page `sql-error-conditions-sqlstates.md`.
### Why are the changes needed?
- Eliminate misunderstandings
https://spark.apache.org/docs/latest/sql-error-conditions-sqlstates.html
<img width="1252" alt="image" src="https://github.com/apache/spark/assets/15246973/287a3699-c0e8-4416-a6b1-4651388f05d0">
- The file `shared/replacements.md` does not exist in the `spark codebase`.
### Does this PR introduce _any_ user-facing change?
Yes, only for web page.
### How was this patch tested?
Manually test.
### 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] Remove the extra text on page sql-error-conditions-sqlstates [spark]
Posted by "yaooqinn (via GitHub)" <gi...@apache.org>.
yaooqinn closed pull request #45469: [MINOR][DOCS] Remove the extra text on page sql-error-conditions-sqlstates
URL: https://github.com/apache/spark/pull/45469
--
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] Remove the extra text on page sql-error-conditions-sqlstates [spark]
Posted by "panbingkun (via GitHub)" <gi...@apache.org>.
panbingkun commented on PR #45469:
URL: https://github.com/apache/spark/pull/45469#issuecomment-1989938050
The pr related to spark-website is here: https://github.com/apache/spark-website/pull/508
--
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] Remove the extra text on page sql-error-conditions-sqlstates [spark]
Posted by "yaooqinn (via GitHub)" <gi...@apache.org>.
yaooqinn commented on PR #45469:
URL: https://github.com/apache/spark/pull/45469#issuecomment-1989889832
Merged to master.
Thank you all.
--
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] Remove the extra text on page sql-error-conditions-sqlstates [spark]
Posted by "panbingkun (via GitHub)" <gi...@apache.org>.
panbingkun commented on PR #45469:
URL: https://github.com/apache/spark/pull/45469#issuecomment-1989791290
This issue has existed since version `3.4.0`. After this PR, I will submit `a patch` to fix the doc in `spark-website`.
https://spark.apache.org/docs/3.4.0/sql-error-conditions-sqlstates.html
https://spark.apache.org/docs/3.4.1/sql-error-conditions-sqlstates.html
https://spark.apache.org/docs/3.4.2/sql-error-conditions-sqlstates.html
https://spark.apache.org/docs/3.5.0/sql-error-conditions-sqlstates.html
https://spark.apache.org/docs/3.5.1/sql-error-conditions-sqlstates.html
--
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] Remove the extra text on page sql-error-conditions-sqlstates [spark]
Posted by "panbingkun (via GitHub)" <gi...@apache.org>.
panbingkun commented on PR #45469:
URL: https://github.com/apache/spark/pull/45469#issuecomment-1989795305
cc @itholic @zhengruifeng @HyukjinKwon
--
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