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

change additional buildpack to java #1773

Merged
merged 2 commits into from
May 25, 2023
Merged

change additional buildpack to java #1773

merged 2 commits into from
May 25, 2023

Conversation

edithwuly
Copy link
Contributor

Summary

Change additional buildpack for benchmark to java.

Output

The time spent on adding additional buildpack for pack build will be more apparent.

Before

Old additional buildpack is too small to show the difference after some optimizations are made.

After

Java buildpack is much bigger. Hopefully, we can see obvious drop in benchmark graphs.

Documentation

  • Should this change be documented?
    • Yes, see #___
    • No

Related

Resolves #___

@edithwuly edithwuly requested review from a team as code owners May 22, 2023 06:31
@github-actions github-actions bot added this to the 0.30.0 milestone May 22, 2023
@github-actions github-actions bot added type/chore Issue that requests non-user facing changes. type/enhancement Issue that requests a new feature or improvement. labels May 22, 2023
natalieparellano added a commit that referenced this pull request May 23, 2023
The previous benchmarks used a buildpack that was small in size,
hence any performance improvements would be hard to spot.

#1773 will switch the buildpack to a large buildpack,
thus making the benchmark more useful.

Signed-off-by: Natalie Arellano <[email protected]>
@natalieparellano
Copy link
Member

@buildpacks/platform-maintainers could you please merge this PR first: #1777

Copy link
Member

@jjbustamante jjbustamante left a comment

Choose a reason for hiding this comment

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

LGTM

@jkutner jkutner enabled auto-merge May 25, 2023 19:55
@jkutner jkutner merged commit 52902b0 into buildpacks:main May 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/chore Issue that requests non-user facing changes. type/enhancement Issue that requests a new feature or improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants