Skip to content

Revert parallelism in CI IT build#3164

Merged
wu-sheng merged 4 commits intoapache:masterfrom
kezhenxu94:test/java-doc
Jul 25, 2019
Merged

Revert parallelism in CI IT build#3164
wu-sheng merged 4 commits intoapache:masterfrom
kezhenxu94:test/java-doc

Conversation

@kezhenxu94
Copy link
Member

Please answer these questions before submitting pull request

  • Why submit this pull request?
  • Bug fix
  • New feature provided
  • Improve performance

A recent build job shows that the javadoc:javadoc is causing issues, we don't host JavaDoc pages and this task is a waste of time/resources, so I'm just removing it

image

@kezhenxu94 kezhenxu94 added the test Test requirements about performance, feature or before release. label Jul 25, 2019
@kezhenxu94 kezhenxu94 added this to the 6.3.0 milestone Jul 25, 2019
@kezhenxu94 kezhenxu94 requested a review from wu-sheng July 25, 2019 01:45
@wu-sheng
Copy link
Member

We must build. Apache release policy. Doc must be built successfully.

@wu-sheng wu-sheng closed this Jul 25, 2019
@wu-sheng wu-sheng added the wontfix This will not be worked on label Jul 25, 2019
@kezhenxu94 kezhenxu94 reopened this Jul 25, 2019
wu-sheng
wu-sheng previously approved these changes Jul 25, 2019
Copy link
Member

@wu-sheng wu-sheng left a comment

Choose a reason for hiding this comment

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

LGTM. Update PR title please.

@kezhenxu94 kezhenxu94 changed the title Don't build JavaDoc since we don't host JavaDoc pages Revert parallelism in JavaDoc build job Jul 25, 2019
@wu-sheng
Copy link
Member

@kezhenxu94 Check the error log, look like even build fails occasionally.

@kezhenxu94 kezhenxu94 changed the title Revert parallelism in JavaDoc build job Revert parallelism in CI IT build Jul 25, 2019
Copy link
Member

@wu-sheng wu-sheng left a comment

Choose a reason for hiding this comment

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

LGTM. Let's revert it.

@wu-sheng wu-sheng merged commit c11b9f1 into apache:master Jul 25, 2019
@kezhenxu94 kezhenxu94 deleted the test/java-doc branch July 25, 2019 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test Test requirements about performance, feature or before release. wontfix This will not be worked on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants