Skip to content

Port dart cataloger to new generic cataloger pattern#1285

Merged
wagoodman merged 1 commit intomainfrom
update-dart-generic-cataloger
Oct 24, 2022
Merged

Port dart cataloger to new generic cataloger pattern#1285
wagoodman merged 1 commit intomainfrom
update-dart-generic-cataloger

Conversation

@wagoodman
Copy link
Copy Markdown
Contributor

@wagoodman wagoodman commented Oct 24, 2022

Follow up to #1281 , porting the Dart cataloger to the new generic.Cataloger

(note: base branch will chain to main once #1284 merges and this is rebased)

@wagoodman wagoodman requested a review from a team October 24, 2022 19:57
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Oct 24, 2022

Benchmark Test Results

Benchmark results from the latest changes vs base branch
name                                                       time/op
ImagePackageCatalogers/alpmdb-cataloger-2                  11.2ms ± 0%
ImagePackageCatalogers/ruby-gemspec-cataloger-2            1.25ms ± 1%
ImagePackageCatalogers/python-package-cataloger-2          3.17ms ± 0%
ImagePackageCatalogers/php-composer-installed-cataloger-2  1.01ms ± 1%
ImagePackageCatalogers/javascript-package-cataloger-2       707µs ± 0%
ImagePackageCatalogers/dpkgdb-cataloger-2                   820µs ± 0%
ImagePackageCatalogers/rpm-db-cataloger-2                  1.19ms ± 0%
ImagePackageCatalogers/java-cataloger-2                    14.1ms ± 1%
ImagePackageCatalogers/apkdb-cataloger-2                   1.14ms ± 1%
ImagePackageCatalogers/go-module-binary-cataloger-2        2.11µs ± 1%
ImagePackageCatalogers/dotnet-deps-cataloger-2             1.29ms ± 0%
ImagePackageCatalogers/portage-cataloger-2                  652µs ± 0%

name                                                       alloc/op
ImagePackageCatalogers/alpmdb-cataloger-2                  5.26MB ± 0%
ImagePackageCatalogers/ruby-gemspec-cataloger-2             202kB ± 0%
ImagePackageCatalogers/python-package-cataloger-2           944kB ± 0%
ImagePackageCatalogers/php-composer-installed-cataloger-2   214kB ± 0%
ImagePackageCatalogers/javascript-package-cataloger-2       158kB ± 0%
ImagePackageCatalogers/dpkgdb-cataloger-2                   203kB ± 0%
ImagePackageCatalogers/rpm-db-cataloger-2                   302kB ± 0%
ImagePackageCatalogers/java-cataloger-2                    3.44MB ± 0%
ImagePackageCatalogers/apkdb-cataloger-2                   1.25MB ± 0%
ImagePackageCatalogers/go-module-binary-cataloger-2          672B ± 0%
ImagePackageCatalogers/dotnet-deps-cataloger-2              367kB ± 0%
ImagePackageCatalogers/portage-cataloger-2                  136kB ± 0%

name                                                       allocs/op
ImagePackageCatalogers/alpmdb-cataloger-2                   85.7k ± 0%
ImagePackageCatalogers/ruby-gemspec-cataloger-2             4.24k ± 0%
ImagePackageCatalogers/python-package-cataloger-2           16.5k ± 0%
ImagePackageCatalogers/php-composer-installed-cataloger-2   5.50k ± 0%
ImagePackageCatalogers/javascript-package-cataloger-2       3.31k ± 0%
ImagePackageCatalogers/dpkgdb-cataloger-2                   4.57k ± 0%
ImagePackageCatalogers/rpm-db-cataloger-2                   8.11k ± 0%
ImagePackageCatalogers/java-cataloger-2                     57.5k ± 0%
ImagePackageCatalogers/apkdb-cataloger-2                    5.39k ± 0%
ImagePackageCatalogers/go-module-binary-cataloger-2          15.0 ± 0%
ImagePackageCatalogers/dotnet-deps-cataloger-2              7.15k ± 0%
ImagePackageCatalogers/portage-cataloger-2                  3.58k ± 0%

Base automatically changed from update-conan-generic-cataloger to main October 24, 2022 20:11
Signed-off-by: Alex Goodman <alex.goodman@anchore.com>
@wagoodman wagoodman force-pushed the update-dart-generic-cataloger branch from 2cddf5b to 1079a52 Compare October 24, 2022 20:14
@wagoodman wagoodman enabled auto-merge (squash) October 24, 2022 20:14
@wagoodman wagoodman merged commit fbdde6d into main Oct 24, 2022
@wagoodman wagoodman deleted the update-dart-generic-cataloger branch October 24, 2022 20:37
GijsCalis pushed a commit to GijsCalis/syft that referenced this pull request Feb 19, 2024
Signed-off-by: Alex Goodman <alex.goodman@anchore.com>

Signed-off-by: Alex Goodman <alex.goodman@anchore.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants