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

Persist analysis files on the background & in parallel #419

Merged
merged 1 commit into from
Apr 6, 2018

Conversation

jvican
Copy link
Contributor

@jvican jvican commented Apr 6, 2018

+ update to latest nailgun.

Previously, bloop would persist the analysis file on shutDown.
However, this hook has been removed upstream in
facebookarchive/nailgun#131.

As a result, we take another (more resilient) strategy to persist the
analysis file. We persist analysis file on the io pool after every
Interpreter execution, without blocking the return of the request.

All the logs are redirected to the nailgun server output.

Previously, bloop would persist the analysis file on `shutDown`.
However, this hook has been removed upstream in
facebookarchive/nailgun#131.

As a result, we take another (more resilient) strategy to persist the
analysis file. We persist analysis file on the io pool after every
`Interpreter` execution, without blocking the return of the request.

All the logs are redirected to the nailgun server output.
@jvican jvican added enhancement build server Any issue or pull request that has to do with hot compilers or BSP. nailgun task / compile labels Apr 6, 2018
@jvican jvican changed the title Persist analysis files on the background Persist analysis files on the background & in parallel Apr 6, 2018
@jvican jvican merged commit 6b66ef0 into master Apr 6, 2018
@tgodzik tgodzik deleted the topic/json+update-nailgun branch September 7, 2021 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build server Any issue or pull request that has to do with hot compilers or BSP. enhancement nailgun task / compile
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant