aws / aws/aws-cdk-cli

(cli/core): concurrent asset builds for ILocalBundling.tryBundle()

Open
#1,200 0 comments 3 reactions 0 assignees View on GitHub
aws-cdk feature-request p1
Dominant language
TypeScript
Stars
105
Forks
122
Avg merge
1d 17h
Merged PRs (30d)
71

Description

### Describe the feature

In this PR https://github.com/aws/aws-cdk-cli/pull/983 - were introduced concurrent asset builds for Docker builds, but we need the same for local

comment from previous PR

You're right that --asset-build-concurrency only applies to Docker-based asset builds in the CLI's work graph, not to ILocalBundling.

Deferring local bundling to the work graph is a very appealing idea. However, ILocalBundling.tryBundle() is called synchronously during synthesis inside the CDK app process, which is a completely separate phase from the CLI's work graph execution. This would require changes on both the framework side (aws-cdk-lib) and the CLI side, which is a significant architectural change beyond the scope of this PR.

It might be worth opening a separate feature request issue for that so the maintainers can weigh in on the approach. In the meantime, a note in the documentation clarifying that this flag applies to Docker-based builds only could also help avoid confusion.

### Use Case

I have 1xxx of golang lambdas builded localy - I would like to use the same --asset-build-concurrency 4 flag for these local builds

### Proposed Solution

_No response_

### Other Information

_No response_

### Acknowledgements

- [ ] I may be able to implement this feature request
- [ ] This feature might incur a breaking change

### CDK version used

v2.1109.0

### Environment details (OS name and version, etc.)

Fedora 44

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.