Skip to content

[Docs]Fix website io connector link 404 #5347

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.

Sign up for GitHub

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

Jump to bottom
Merged
merlimat merged 5 commits into from
Oct 24, 2019
Merged

[Docs]Fix website io connector link 404 #5347

merlimat merged 5 commits into from
Oct 24, 2019

Conversation

@tuteng
Copy link
Member

@tuteng tuteng commented Oct 9, 2019

Motivation

Currently, the old version of the connector document is incorrect. The new version of the connector document is not compatible with the old version, resulting in an HTTP 404 response.

Modifications

  • Documentation for older versions http link of connectors remains unchanged.
  • Add document of old versions.

Verifying this change

Local Test Pass

@aahmed-se
Copy link
Contributor

aahmed-se commented Oct 10, 2019

rerun cpp tests

@aahmed-se
Copy link
Contributor

aahmed-se commented Oct 10, 2019

run java8 tests

1 similar comment
@aahmed-se
Copy link
Contributor

aahmed-se commented Oct 10, 2019

run java8 tests

@aahmed-se
Copy link
Contributor

aahmed-se commented Oct 10, 2019

@tuteng rebase this
run java8 tests

@merlimat merlimat added the doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. label Oct 10, 2019
@merlimat merlimat added this to the 2.5.0 milestone Oct 10, 2019
@aahmed-se
Copy link
Contributor

aahmed-se commented Oct 10, 2019

run java8 tests
run cpp tests

@tuteng
Copy link
Member Author

tuteng commented Oct 10, 2019

run java8 tests

2 similar comments
@merlimat
Copy link
Contributor

merlimat commented Oct 10, 2019

run java8 tests

@tuteng
Copy link
Member Author

tuteng commented Oct 11, 2019

run java8 tests

@tuteng tuteng requested a review from jiazhai October 11, 2019 00:19
@tuteng tuteng self-assigned this Oct 11, 2019
@tuteng tuteng requested a review from sijie October 11, 2019 00:20
@tuteng
Copy link
Member Author

tuteng commented Oct 11, 2019

run java8 tests

1 similar comment
@tuteng
Copy link
Member Author

tuteng commented Oct 12, 2019

run java8 tests

@tuteng tuteng requested a review from merlimat October 18, 2019 02:02
@tuteng
Copy link
Member Author

tuteng commented Oct 22, 2019

run java8 tests

@tuteng
Copy link
Member Author

tuteng commented Oct 22, 2019

run cpp tests

@tuteng
Copy link
Member Author

tuteng commented Oct 23, 2019

retest this please

@tuteng
Copy link
Member Author

tuteng commented Oct 23, 2019

run java8 tests

@tuteng tuteng mentioned this pull request Oct 24, 2019
Closed
@merlimat merlimat merged commit a71ac20 into apache:master Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sijie sijie sijie approved these changes

@jiazhai jiazhai Awaiting requested review from jiazhai

@merlimat merlimat Awaiting requested review from merlimat

Assignees

@tuteng tuteng

Labels

doc Your PR contains doc changes, no matter whether the changes are in markdown or code files.

Projects

None yet

Milestone

2.5.0

Development

Successfully merging this pull request may close these issues.

4 participants

@tuteng @aahmed-se @merlimat @sijie