Skip to content

Conversation

@mark-vieira
Copy link
Contributor

This PR makes some updates to the IDE compiler options when importing the project to account for some changes in IntelliJ 2020.2. In this update the IDE is more aggressive in forking compilation processes under certain circumstances. Thes forked processes share a slice of the total build process heap size, which is 700MB by default. With several forked compilers, these each only receive 256MB, which isn't sufficient for compiling the largest modules, such as server. We bump this to 2G to give ample room for parallel and forked compilation in the IDE. We also tweak some other settings to minimize the amount of forking as well as disable annotation post-processing, which isn't needed.

@mark-vieira mark-vieira added the :Delivery/Build Build or test infrastructure label Jul 28, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (:Core/Infra/Build)

@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label Jul 28, 2020
@mark-vieira mark-vieira requested a review from rjernst July 28, 2020 17:34
Copy link
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

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

LGTM

@mark-vieira
Copy link
Contributor Author

@elasticmachine update branch

@mark-vieira
Copy link
Contributor Author

@elasticmachine update branch

@mark-vieira mark-vieira merged commit 57c25cd into master Jul 28, 2020
@mark-vieira mark-vieira deleted the intellij-2020.2-fixes branch July 28, 2020 19:39
mark-vieira added a commit that referenced this pull request Jul 28, 2020
mark-vieira added a commit that referenced this pull request Jul 28, 2020
mark-vieira added a commit that referenced this pull request Jul 28, 2020
mark-vieira added a commit that referenced this pull request Jul 28, 2020
@mark-vieira mark-vieira added Team:Delivery Meta label for Delivery team and removed Team:Core/Infra Meta label for core/infra team labels Nov 11, 2020
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.

5 participants