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

Set MessageAttribute to "No" in SendMessageBatchRequestEntry #917

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

iglebov
Copy link
Contributor

@iglebov iglebov commented Mar 15, 2025

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=ru

Description of changes:

  1. Set MessageAttribute to "No" in SendMessageBatchRequestEntry

@iglebov
Copy link
Contributor Author

iglebov commented Mar 15, 2025

Fixes this issue:
#708

@yfm-team
Copy link

Все готово!
Commit: 85e9da3
Проверьте результат: RU.
Check the result: EN.

@vnkiryushkin vnkiryushkin added the message-queue Yandex Message Queue label Mar 17, 2025
@@ -6,7 +6,7 @@ Contains the details of a message being sent and its ID. For more information ab
----- | ----- | ----- | -----
| `DelaySeconds` | **integer** | No | The number of seconds to delay sending a message. |
| `Id` | **string** | Yes | ID of the message in the list. |
| `MessageAttribute` | **string** | Yes | Message attributes: name, type, and value. |
| `MessageAttribute` | **string** | No | Message attributes: name, type, and value. |
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ничего не понятно: в каком формате это всё запаковывается в строку? что происходит, когда оно не указано, раз становится необязательным?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@blinkov

Ivan, добрый день!

Согласен, мне тоже непонятно)

Создал MR в соответствии с issue.
Если он некорректный, то могу закрыть.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

я не знаю корректный он или нет — надо искать кто ответственный за эту фичу и с ним разбираться как что-то понятное соорудить

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
message-queue Yandex Message Queue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants