Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MINOR][ML][DOCS] Fix sql data types link in the ml-pipeline page #36633

Closed
wants to merge 1 commit into from

Conversation

yaooqinn
Copy link
Member

What changes were proposed in this pull request?

image

Spark SQL datatype reference - https://spark.apache.org/docs/latest/sql-reference.html#data-types is invalid and it shall be Spark SQL datatype reference - https://spark.apache.org/docs/latest/sql-ref-datatypes.html
https://spark.apache.org/docs/latest/ml-pipeline.html#dataframe

Why are the changes needed?

doc fix

Does this PR introduce any user-facing change?

no

How was this patch tested?

bundle exec jekyll serve

@github-actions github-actions bot added the DOCS label May 23, 2022
@huaxingao huaxingao closed this in de73753 May 23, 2022
huaxingao pushed a commit that referenced this pull request May 23, 2022
### What changes were proposed in this pull request?

<img width="939" alt="image" src="https://user-images.githubusercontent.com/8326978/169767919-6c48554c-87ff-4d40-a47d-ec4da0c993f7.png">

[Spark SQL datatype reference](https://spark.apache.org/docs/latest/sql-reference.html#data-types) - `https://spark.apache.org/docs/latest/sql-reference.html#data-types` is invalid and it shall be [Spark SQL datatype reference](https://spark.apache.org/docs/latest/sql-ref-datatypes.html) - `https://spark.apache.org/docs/latest/sql-ref-datatypes.html`
https://spark.apache.org/docs/latest/ml-pipeline.html#dataframe

### Why are the changes needed?

doc fix

### Does this PR introduce _any_ user-facing change?

no

### How was this patch tested?

`bundle exec jekyll serve`

Closes #36633 from yaooqinn/minor.

Authored-by: Kent Yao <yao@apache.org>
Signed-off-by: huaxingao <huaxin_gao@apple.com>
(cherry picked from commit de73753)
Signed-off-by: huaxingao <huaxin_gao@apple.com>
huaxingao pushed a commit that referenced this pull request May 23, 2022
### What changes were proposed in this pull request?

<img width="939" alt="image" src="https://user-images.githubusercontent.com/8326978/169767919-6c48554c-87ff-4d40-a47d-ec4da0c993f7.png">

[Spark SQL datatype reference](https://spark.apache.org/docs/latest/sql-reference.html#data-types) - `https://spark.apache.org/docs/latest/sql-reference.html#data-types` is invalid and it shall be [Spark SQL datatype reference](https://spark.apache.org/docs/latest/sql-ref-datatypes.html) - `https://spark.apache.org/docs/latest/sql-ref-datatypes.html`
https://spark.apache.org/docs/latest/ml-pipeline.html#dataframe

### Why are the changes needed?

doc fix

### Does this PR introduce _any_ user-facing change?

no

### How was this patch tested?

`bundle exec jekyll serve`

Closes #36633 from yaooqinn/minor.

Authored-by: Kent Yao <yao@apache.org>
Signed-off-by: huaxingao <huaxin_gao@apple.com>
(cherry picked from commit de73753)
Signed-off-by: huaxingao <huaxin_gao@apple.com>
@huaxingao
Copy link
Contributor

Thanks! Merged to master/3.3/3.2. I somehow had problem to merge to 3.1 so I stopped there. I think the problem is also in 3.0 and 3.1.

kazuyukitanimura pushed a commit to kazuyukitanimura/spark that referenced this pull request Aug 10, 2022
### What changes were proposed in this pull request?

<img width="939" alt="image" src="https://user-images.githubusercontent.com/8326978/169767919-6c48554c-87ff-4d40-a47d-ec4da0c993f7.png">

[Spark SQL datatype reference](https://spark.apache.org/docs/latest/sql-reference.html#data-types) - `https://spark.apache.org/docs/latest/sql-reference.html#data-types` is invalid and it shall be [Spark SQL datatype reference](https://spark.apache.org/docs/latest/sql-ref-datatypes.html) - `https://spark.apache.org/docs/latest/sql-ref-datatypes.html`
https://spark.apache.org/docs/latest/ml-pipeline.html#dataframe

### Why are the changes needed?

doc fix

### Does this PR introduce _any_ user-facing change?

no

### How was this patch tested?

`bundle exec jekyll serve`

Closes apache#36633 from yaooqinn/minor.

Authored-by: Kent Yao <yao@apache.org>
Signed-off-by: huaxingao <huaxin_gao@apple.com>
(cherry picked from commit de73753)
Signed-off-by: huaxingao <huaxin_gao@apple.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants