Skip to content
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

Fix/Fill split info for the first child correctly #2378

Conversation

notimetoname
Copy link
Contributor

@notimetoname notimetoname commented Jun 12, 2023

SDK RC8 does not attach a split ID to the first child in a split chain.

See nspcc-dev/neofs-sdk-go#448 and https://nspcc-dev.github.io/neofs-node/191/#suites/beb587d64f4c3e0a7b76bb9b5b406a00/57ccae6fb58af59b.

@notimetoname notimetoname added the bug Something isn't working label Jun 12, 2023
@notimetoname notimetoname self-assigned this Jun 12, 2023
SDK RC8 does not attach a split ID to the first child in a split chain.

Signed-off-by: Pavel Karpy <carpawell@morphbits.io>
@notimetoname notimetoname force-pushed the fix/missing-split-id-in-first-child branch from 594fb04 to 9b23ffe Compare June 12, 2023 15:41
@codecov
Copy link

codecov bot commented Jun 12, 2023

Codecov Report

Merging #2378 (2742fc5) into master (82b9409) will increase coverage by 0.00%.
The diff coverage is n/a.

❗ Current head 2742fc5 differs from pull request most recent head 9b23ffe. Consider uploading reports for the commit 9b23ffe to get more accurate results

@@           Coverage Diff           @@
##           master    #2378   +/-   ##
=======================================
  Coverage   30.04%   30.05%           
=======================================
  Files         398      398           
  Lines       29806    29806           
=======================================
+ Hits         8955     8957    +2     
+ Misses      20126    20124    -2     
  Partials      725      725           

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Member

@roman-khimov roman-khimov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess with this one you've almost reimplemented the slicer in the node anyway...

@roman-khimov roman-khimov merged commit a70bbe9 into nspcc-dev:master Jun 13, 2023
@notimetoname notimetoname deleted the fix/missing-split-id-in-first-child branch June 13, 2023 07:35
@notimetoname notimetoname mentioned this pull request Jun 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants