[FEATURE] broker.Publish доп метаданные сообщения #296

Closed
opened 2024-02-25 23:51:38 +03:00 by vtolstov · 6 comments
Owner

нужно сделать поддержку опции option.Metadata(kv ...string) в options.go (сделать switch/case для типа аргумента, что там либо мапа либо массив строк). Если массив строк то нечетное это ключ, а четное это значение.

эти метаданные должны попасть в метаданные сообщения в брокере.
Данную правку делать только в мастер ветке
https://git.unistack.org/unistack-org/micro/src/branch/master/options/options.go#L153

нужно сделать поддержку опции option.Metadata(kv ...string) в options.go (сделать switch/case для типа аргумента, что там либо мапа либо массив строк). Если массив строк то нечетное это ключ, а четное это значение. эти метаданные должны попасть в метаданные сообщения в брокере. Данную правку делать только в мастер ветке https://git.unistack.org/unistack-org/micro/src/branch/master/options/options.go#L153
devstigneev was assigned by vtolstov 2024-02-25 23:51:44 +03:00
devstigneev added a new dependency 2024-02-28 17:15:57 +03:00
Member

Новый метод, или старый изменяем?
Тк на вход массив строк или мапа, то пока что аргумент - any

Новый метод, или старый изменяем? Тк на вход массив строк или мапа, то пока что аргумент - any
Author
Owner

Старый метод меняй в соответствии с новым вариантом.

Старый метод меняй в соответствии с новым вариантом.
vtolstov removed a dependency 2024-02-28 23:52:24 +03:00
devstigneev added a new dependency 2024-02-29 00:20:23 +03:00
Member

с вариативным вариантом выглядеть стало как то костыльно)

с вариативным вариантом выглядеть стало как то костыльно)
vtolstov removed a dependency 2024-02-29 01:27:23 +03:00
Author
Owner

Почему? Одно из преимуществ что тебе не нужно делать новые аллокации временные для мапы =)

Почему? Одно из преимуществ что тебе не нужно делать новые аллокации временные для мапы =)
Member

ну, в плане не оч аккуратно и несколько свитчтайпов)

ну, в плане не оч аккуратно и несколько свитчтайпов)
Author
Owner

ну, в плане не оч аккуратно и несколько свитчтайпов)

норм =) зато вроде быстро и понятно.

> ну, в плане не оч аккуратно и несколько свитчтайпов) норм =) зато вроде быстро и понятно.
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 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: unistack-org/micro#296
No description provided.