Respect TTL for EC put #1288

dstepanov-yadro · 2024-08-01T13:15:52Z

dstepanov-yadro commented

2024-08-01 13:15:52 +00:00

Scenario:

EC 2.1 CBF 1 SELECT 4 FROM * container, netmap contains 4 nodes, but one node (node 2) is off
Client puts object to node 1, node 1 splits object to 3 chunks
Node 1 tries to save chunk 0 to node 2, gets error, and tries to save chunk 0 to node 4.
Node 4 tries to save chunk 0 to node 2 again, gets error, and saves chunk 0 locally or tries to save chunk 0 to node 3.

Now node takes into account if node gets prepared EC part. It should work so:

Scenario: 1. `EC 2.1 CBF 1 SELECT 4 FROM *` container, netmap contains 4 nodes, but one node (`node 2`) is off 2. Client puts object to `node 1`, `node 1` splits object to 3 chunks 3. `Node 1` tries to save `chunk 0` to `node 2`, gets error, and tries to save `chunk 0` to `node 4`. 4. `Node 4` tries to save `chunk 0` to `node 2` again, gets error, and saves `chunk 0` locally or tries to save `chunk 0` to `node 3`. Now node takes into account if node gets prepared EC part. It should work so: ![image](/attachments/5eb7ccbf-e97f-4ae1-9c36-d3ab3fc778d0)

image.png

56 KiB

dstepanov-yadro force-pushed fix/ec_put_respect_ttl from 8f782503b0 to e4179dac8f

2024-08-01 13:17:40 +00:00

Compare

fyrchik reviewed 2024-08-01 13:26:50 +00:00

pkg/services/object/put/ec.go Outdated

					
				@ -238,6 +251,12 @@ func (e *ecWriter) writePart(ctx context.Context, obj *objectSDK.Object, partIdx

					partVisited := make([]bool, len(nodes))

					partVisited[partIdx%len(nodes)] = true

					select {

fyrchik commented

2024-08-01 13:26:48 +00:00

Same here

dstepanov-yadro commented

2024-08-01 13:29:06 +00:00

Fixed

fyrchik marked this conversation as resolved

pkg/services/object/put/ec.go Outdated

					
				@ -263,6 +282,12 @@ func (e *ecWriter) writePart(ctx context.Context, obj *objectSDK.Object, partIdx

						partVisited[idx] = true

					}

					select {