Re: [PR] [MINOR][DOCS] Remove the extra text on page sql-error-conditions-sqlstates [spark]
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-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
Re: [PR] [MINOR][DOCS] Remove the extra text on page sql-error-conditions-sqlstates [spark]
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-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
Re: [PR] [MINOR][DOCS] Remove the extra text on page sql-error-conditions-sqlstates [spark]
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-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
Re: [PR] [MINOR][DOCS] Remove the extra text on page sql-error-conditions-sqlstates [spark]
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-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
Re: [PR] [MINOR][DOCS] Remove the extra text on page sql-error-conditions-sqlstates [spark]
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-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[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 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-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org