Parallel upload - Githubissues

Hitenjain14 commented 7 months ago

Changes

Fixes

Tests

Tasks to complete before merging PR:

[ ] Ensure system tests are passing. If not Run them manually to check for any regressions :clipboard:
[ ] Do any new system tests need added to test this change? do any existing system tests need updated? If so create a PR at 0chain/system_test
[ ] Merge your system tests PR to master AFTER merging this PR

Associated PRs (Link as appropriate):

0chain:
gosdk:
system_test:
zboxcli:
zwalletcli:
Other: ...

codecov-commenter commented 7 months ago

Codecov Report

Attention: 192 lines in your changes are missing coverage. Please review.

Comparison is base (1d81b05) 14.40% compared to head (d9f2fbc) 14.95%.

Files	Patch %	Lines
...go/0chain.net/blobbercore/allocation/connection.go	9.72%	63 Missing and 2 partials :warning:
...ode/go/0chain.net/blobbercore/filestore/storage.go	26.82%	28 Missing and 2 partials :warning:
...ain.net/blobbercore/handler/file_command_update.go	0.00%	25 Missing :warning:
...chain.net/blobbercore/filestore/tree_validation.go	63.93%	18 Missing and 4 partials :warning:
...et/blobbercore/handler/object_operation_handler.go	25.00%	15 Missing and 3 partials :warning:
...ain.net/blobbercore/handler/file_command_upload.go	46.66%	13 Missing and 3 partials :warning:
...ain.net/blobbercore/allocation/allocationchange.go	0.00%	10 Missing :warning:
code/go/0chain.net/core/common/handler.go	50.00%	3 Missing :warning:
...ain.net/blobbercore/handler/file_command_delete.go	0.00%	2 Missing :warning:
code/go/0chain.net/blobbercore/handler/worker.go	0.00%	1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## sprint-1.13 #1374 +/- ## =============================================== + Coverage 14.40% 14.95% +0.54% =============================================== Files 177 178 +1 Lines 19859 19854 -5 =============================================== + Hits 2861 2969 +108 + Misses 16642 16520 -122 - Partials 356 365 +9 ``` | [Flag](https://app.codecov.io/gh/0chain/blobber/pull/1374/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=0chain) | Coverage Δ | | |---|---|---| | [Unit-Tests](https://app.codecov.io/gh/0chain/blobber/pull/1374/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=0chain) | `14.95% <42.68%> (+0.54%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=0chain#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

service-0chain commented 7 months ago

Manual system tests [cancelled] with the following config	config	value
tests skipped	false
system-tests
gosdk	NOT OVERRIDDEN
zboxcli
zwalletcli
0chain
blobber	feat/parallel-upload
authorizer
0box
0dns