-
Notifications
You must be signed in to change notification settings - Fork 103
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
make test
Failures
#145
Comments
tdruez
added a commit
that referenced
this issue
Apr 21, 2021
…pool processes #145 Signed-off-by: Thomas Druez <tdruez@nexb.com>
@AyanSinhaMahapatra could you run the tests again using the latests code from https://github.com/nexB/scancode.io/compare/145-multiprocessing
Let me know if we get better results. |
tdruez
added a commit
that referenced
this issue
Apr 21, 2021
Signed-off-by: Thomas Druez <tdruez@nexb.com>
tdruez
added a commit
that referenced
this issue
Apr 21, 2021
Signed-off-by: Thomas Druez <tdruez@nexb.com>
@tdruez This solves the test failure issue. Thanks. |
tdruez
added a commit
that referenced
this issue
Apr 22, 2021
…s pipe #145 Signed-off-by: Thomas Druez <tdruez@nexb.com>
tdruez
added a commit
that referenced
this issue
Apr 22, 2021
Signed-off-by: Thomas Druez <tdruez@nexb.com>
tdruez
added a commit
that referenced
this issue
Apr 22, 2021
…#145 This has a negative impact on performances Signed-off-by: Thomas Druez <tdruez@nexb.com>
tdruez
added a commit
that referenced
this issue
Apr 22, 2021
… the sub-processes #145 (#159) * Refactor scan_for_files to extract the database transaction from the pool processes #145 Signed-off-by: Thomas Druez <tdruez@nexb.com> * Refine logging message formatting #145 Signed-off-by: Thomas Druez <tdruez@nexb.com> * Log the scan_for_files progress to the database Run object #145 Signed-off-by: Thomas Druez <tdruez@nexb.com> * Apply multiprocessing refactoring to the scan_for_application_packages pipe #145 Signed-off-by: Thomas Druez <tdruez@nexb.com> * Add entry in the CHANGELOG #145 Signed-off-by: Thomas Druez <tdruez@nexb.com> * Remove the logging to the database for scancode pipe scanning function #145 This has a negative impact on performances Signed-off-by: Thomas Druez <tdruez@nexb.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
On latest
main
and a fresh configureTraceback:
Though running the scan_codebase pipeline works perfectly on a couple of files.
The text was updated successfully, but these errors were encountered: