nspcc-dev / neofs-node

NeoFS is a decentralized distributed object storage integrated with the Neo blockchain
https://fs.neo.org
GNU General Public License v3.0
32 stars 38 forks source link

Simplify engine interface #3008

Closed roman-khimov closed 1 week ago

roman-khimov commented 1 week ago

Mostly similar to #3001, we have a number of Prm/Res structures that make it harder to used engine and trace things going on. But there are some functional changes here as well and also things that go beyond Prm/Res. Overall we get rid of many pointless conversions.

codecov[bot] commented 1 week ago

Codecov Report

Attention: Patch coverage is 52.55814% with 102 lines in your changes missing coverage. Please review.

Project coverage is 23.01%. Comparing base (007e992) to head (e482dbf). Report is 17 commits behind head on master.

Files with missing lines Patch % Lines
pkg/local_object_storage/engine/container.go 0.00% 27 Missing :warning:
pkg/local_object_storage/engine/select.go 44.44% 14 Missing and 1 partial :warning:
pkg/services/object/get/util.go 0.00% 7 Missing :warning:
pkg/local_object_storage/engine/evacuate.go 70.00% 5 Missing and 1 partial :warning:
pkg/services/control/server/list_objects.go 0.00% 6 Missing :warning:
cmd/neofs-lens/internal/storage/list.go 0.00% 5 Missing :warning:
pkg/local_object_storage/engine/range.go 80.00% 4 Missing and 1 partial :warning:
pkg/local_object_storage/engine/head.go 77.77% 3 Missing and 1 partial :warning:
pkg/local_object_storage/engine/writecache.go 0.00% 4 Missing :warning:
cmd/neofs-node/object.go 0.00% 3 Missing :warning:
... and 13 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #3008 +/- ## ========================================== - Coverage 23.06% 23.01% -0.06% ========================================== Files 790 790 Lines 58709 58500 -209 ========================================== - Hits 13542 13461 -81 + Misses 44285 44161 -124 + Partials 882 878 -4 ```

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

roman-khimov commented 1 week ago

Greenish: https://rest.fs.neo.org/HXSaMJXk2g8C14ht8HSi7BBaiYZ1HeWh2xnWPGQCg4H6/2083-1731357972/index.html

roman-khimov commented 1 week ago

Green.