Skip to content

[Doc] Add spark load sql statement doc and update manual - #4463

Merged
morningman merged 6 commits into
apache:masterfrom
wyb:spark_load_update_doc
Aug 30, 2020
Merged

[Doc] Add spark load sql statement doc and update manual#4463
morningman merged 6 commits into
apache:masterfrom
wyb:spark_load_update_doc

Conversation

@wyb

@wybwyb commented Aug 27, 2020

Copy link
Copy Markdown
Contributor
  1. add sql statement in dml
  2. update spark load manual

#3433

morningman
morningman reviewed Aug 27, 2020
View reviewed changes

@morningmanmorningman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need also update the sidebar file in docs/.vuepress/sidebar/zh-CN.js and docs/.vuepress/sidebar/en.js

Comment threaddocs/zh-CN/administrator-guide/load-data/spark-load-manual.md
Comment threaddocs/zh-CN/sql-reference/sql-statements/Data Manipulation/SPARK LOAD.md Outdated
@morningmanmorningman added area/spark-load Issues or PRs related to the spark load kind/docs Categorizes issue or PR as related to documentation. labels Aug 27, 2020
@wyb

wyb commented Aug 27, 2020

Copy link
Copy Markdown
ContributorAuthor

You need also update the sidebar file in docs/.vuepress/sidebar/zh-CN.js and docs/.vuepress/sidebar/en.js

Add in zh-CN.js, and i will add en doc after spark load is stable.

WuZixing
WuZixing reviewed Aug 28, 2020
View reviewed changes
Comment threaddocs/zh-CN/sql-reference/sql-statements/Data Manipulation/SPARK LOAD.md Outdated
Comment threaddocs/zh-CN/sql-reference/sql-statements/Data Manipulation/SPARK LOAD.md Outdated
morningman
morningman approved these changes Aug 28, 2020
View reviewed changes

@morningmanmorningman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@morningmanmorningman added the approved Indicates a PR has been approved by one committer. label Aug 28, 2020
@morningman
morningman merged commit ffe696d into apache:masterAug 30, 2020
@yangzhgyangzhg mentioned this pull request Feb 9, 2021
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@morningmanmorningmanmorningman approved these changes
+1 more reviewer
@WuZixingWuZixingWuZixing left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approvedIndicates a PR has been approved by one committer.area/spark-loadIssues or PRs related to the spark loadkind/docsCategorizes issue or PR as related to documentation.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@wyb@morningman@WuZixing