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

Add Multiplex Workers support for Android builds #10773

Closed
nkoroste opened this issue Feb 12, 2020 · 4 comments
Closed

Add Multiplex Workers support for Android builds #10773

nkoroste opened this issue Feb 12, 2020 · 4 comments
Labels
P2 We'll consider working on this in future. (Assignee optional) stale Issues or PRs that are stale (no activity for 30 days) team-Android Issues for Android team type: feature request

Comments

@nkoroste
Copy link
Contributor

Description of the problem / feature request:

From what I can see from https://docs.bazel.build/versions/master/multiplex-worker.html and source code is that Multiplex Workers are only supported for Scala and maybe Java. It would be nice to get this support for Android build actions as well as Android builds tend to be large and slow.

Feature requests: what underlying problem are you trying to solve with this feature?

Improve Android's build performance.

What operating system are you running Bazel on?

Linux & Mac

Have you found anything relevant by searching the web?

@aiuto aiuto added team-Local-Exec Issues and PRs for the Execution (Local) team team-Remote-Exec Issues and PRs for the Execution (Remote) team untriaged labels Feb 12, 2020
@jmmv
Copy link
Contributor

jmmv commented Feb 13, 2020

This issue is specific about changing Android rules/workers to using multiplexing. The infrastructure changes are done, so this has nothing to do with local execution. (Nor remote execution.)

@jmmv jmmv added team-Android Issues for Android team and removed team-Local-Exec Issues and PRs for the Execution (Local) team team-Remote-Exec Issues and PRs for the Execution (Remote) team labels Feb 13, 2020
@larsrc-google
Copy link
Contributor

If your Android workers are written in Java, take a look at WorkRequestHandler, a generic class to handle the request/response and multithreading parts of multiplexing. Of course, the worker itself needs to be thread-safe, including not sharing temp files between threads.

@ahumesky ahumesky added area-EngProd Bazel CI, infrastructure, bootstrapping, release, and distribution tooling P2 We'll consider working on this in future. (Assignee optional) type: feature request and removed untriaged area-EngProd Bazel CI, infrastructure, bootstrapping, release, and distribution tooling labels Dec 3, 2020
@github-actions
Copy link

Thank you for contributing to the Bazel repository! This issue has been marked as stale since it has not had any activity in the last 2+ years. It will be closed in the next 14 days unless any other activity occurs or one of the following labels is added: "not stale", "awaiting-bazeler". Please reach out to the triage team (@bazelbuild/triage) if you think this issue is still relevant or you are interested in getting the issue resolved.

@github-actions github-actions bot added the stale Issues or PRs that are stale (no activity for 30 days) label Apr 13, 2023
@github-actions
Copy link

This issue has been automatically closed due to inactivity. If you're still interested in pursuing this, please reach out to the triage team (@bazelbuild/triage). Thanks!

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Apr 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 We'll consider working on this in future. (Assignee optional) stale Issues or PRs that are stale (no activity for 30 days) team-Android Issues for Android team type: feature request
Projects
None yet
Development

No branches or pull requests

5 participants