Remove response wrapper service #460

fyrchik · 2023-06-21T14:08:42Z

fyrchik commented

2023-06-21 14:08:42 +00:00

See #6, part 1.

fyrchik force-pushed simplify-services-master from f13a3cc0cf to 40eae22109

2023-06-21 14:09:19 +00:00

Compare

fyrchik requested review from storage-core-committers 2023-06-21 14:09:26 +00:00

fyrchik requested review from storage-core-developers 2023-06-21 14:09:30 +00:00

dstepanov-yadro reviewed 2023-06-21 15:12:57 +00:00

pkg/services/object/put/distributed.go

					
				@ -230,3 +224,2 @@

							workerPool = t.localPool

						}

						workerPool, isLocal := t.getWorkerPool(addr.PublicKey())

dstepanov-yadro commented

2023-06-21 15:12:57 +00:00

Looks like isLocal is worker pool property. But actually worker pool type depends on isLocal

Looks like `isLocal` is worker pool property. But actually worker pool type depends on `isLocal`

dstepanov-yadro approved these changes 2023-06-21 15:13:45 +00:00

acid-ant approved these changes 2023-06-22 13:25:09 +00:00

fyrchik merged commit 40eae22109 into master

2023-06-23 08:02:44 +00:00

fyrchik deleted branch simplify-services-master

2023-06-23 08:02:45 +00:00

fyrchik referenced this pull request from a commit

2023-06-23 08:02:45 +00:00

[#460] services/object: Reduce `distibutedTarget` memory footprint

fyrchik referenced this pull request from a commit

2023-06-23 08:02:45 +00:00

[#460] services/util: Remove `HandleUnaryRequest`

fyrchik referenced this pull request from a commit

2023-06-23 08:02:45 +00:00

[#460] services/util: Remove `HandleServerStreamRequest`

fyrchik referenced this pull request from a commit

2023-06-23 08:02:45 +00:00

[#460] services/util: Remove `CreateRequestStreamer`

Sign in to join this conversation.

No reviewers

No labels

No milestone

No project

No assignees

3 participants

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: TrueCloudLab/frostfs-node#460