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

Split object assembly moved to a container nodes (again) #838

Closed
alexvanin opened this issue Sep 22, 2021 · 2 comments
Closed

Split object assembly moved to a container nodes (again) #838

alexvanin opened this issue Sep 22, 2021 · 2 comments
Labels
discussion Open discussion of some problem I4 No visible changes S2 Regular significance U3 Regular

Comments

@alexvanin
Copy link
Contributor

alexvanin commented Sep 22, 2021

Some time ago we decided that nodes which process request, should assemble split objects. There was a config flag to disable forced assembly policy on the node (right now flag is hardcoded).

User <--> [Personal node] <---> [ Container Node, ..., Container Node ]
            Assembles
            Split
            Object

This was achieved by sending new object get requests with RAW flag set. Container nodes return SplitInfoError and personal node is able to build a chain of object and assemble split object.

This has changed with introduction of request forwarder in object service. Now node does not produce new requests if it is not necessary. In described case, personal node re-signs original request and sends it to container nodes. Container nodes that store last object in the chain or linking object are able to build the chain themself and they do assemble required object.

Technically it is totally correct, however it goes against earlier decision to assemble objects on the non container nodes.

/cc @cthulhu-rider @carpawell @realloc @fyrchik

@cthulhu-rider
Copy link
Contributor

Need to explore current implementation.

@roman-khimov
Copy link
Member

Not an issue. Non-container node can't really access container objects.

@roman-khimov roman-khimov closed this as not planned Won't fix, can't repro, duplicate, stale Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion Open discussion of some problem I4 No visible changes S2 Regular significance U3 Regular
Projects
None yet
Development

No branches or pull requests

3 participants