Skip to content

MASSEMBLY-945 - speed improvements#38

Merged
olamy merged 1 commit into
apache:masterfrom
gnodet:speed
Dec 11, 2020
Merged

MASSEMBLY-945 - speed improvements#38
olamy merged 1 commit into
apache:masterfrom
gnodet:speed

Conversation

@gnodet

@gnodet gnodet commented Nov 18, 2020

Copy link
Copy Markdown
Contributor

This PR only makes sense with
codehaus-plexus/plexus-utils#106
codehaus-plexus/plexus-io#33
codehaus-plexus/plexus-archiver#157
apache/maven-common-artifact-filters#15

... but if I add the snapshot upgrades, the checks won't pass anymore ;-)

@elharo

elharo commented Dec 7, 2020

Copy link
Copy Markdown
Contributor

have you measured the improvements? if so, what are the numbers?

@bmarwell

bmarwell commented Dec 7, 2020

Copy link
Copy Markdown

have you measured the improvements? if so, what are the numbers?

Null-length arrays are known to be faster since at least 2013 iirc.

In my opinion measurements are not needed here because of this.

@elharo

elharo commented Dec 7, 2020

Copy link
Copy Markdown
Contributor

Far more often than not, microoptimizations like this result in no measurable impact. All claims of performance improvements need numbers.

@bmarwell bmarwell added the enhancement New feature or request label Dec 10, 2020
@bmarwell

Copy link
Copy Markdown

Far more often than not, microoptimizations like this result in no measurable impact. All claims of performance improvements need numbers.

codehaus-plexus/plexus-io#33 (comment)

@bmarwell bmarwell requested a review from olamy December 10, 2020 12:53
@gnodet

gnodet commented Dec 10, 2020

Copy link
Copy Markdown
Contributor Author

The speed gain may be negligible for this PR, it does not matter much.
However, I'd like the other PR referenced above to get merged, so that we can upgrade to those new versions to have a non negligible performance gain.

@olamy olamy merged commit 9cbdbc4 into apache:master Dec 11, 2020
@eolivelli

Copy link
Copy Markdown
Contributor

+1

@jira-importer

Copy link
Copy Markdown

Resolve #1157

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants