[FEATURE] broker.Publish доп метаданные сообщения #296
Labels
No Label
Kind/Breaking
Kind/Bug
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Security
Kind/Testing
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Reviewed
Confirmed
Reviewed
Duplicate
Reviewed
Invalid
Reviewed
Won't Fix
Status
Abandoned
Status
Blocked
Status
Need More Info
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: unistack-org/micro#296
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
нужно сделать поддержку опции option.Metadata(kv ...string) в options.go (сделать switch/case для типа аргумента, что там либо мапа либо массив строк). Если массив строк то нечетное это ключ, а четное это значение.
эти метаданные должны попасть в метаданные сообщения в брокере.
Данную правку делать только в мастер ветке
https://git.unistack.org/unistack-org/micro/src/branch/master/options/options.go#L153
Новый метод, или старый изменяем?
Тк на вход массив строк или мапа, то пока что аргумент - any
Старый метод меняй в соответствии с новым вариантом.
с вариативным вариантом выглядеть стало как то костыльно)
Почему? Одно из преимуществ что тебе не нужно делать новые аллокации временные для мапы =)
ну, в плане не оч аккуратно и несколько свитчтайпов)
норм =) зато вроде быстро и понятно.