Skip to content

Split send into send-event and send-batch#7

Merged
zmoog merged 2 commits intomainfrom
zmoog/split-send
Jun 4, 2024
Merged

Split send into send-event and send-batch#7
zmoog merged 2 commits intomainfrom
zmoog/split-send

Conversation

@zmoog
Copy link
Copy Markdown
Owner

@zmoog zmoog commented Jun 4, 2024

Motivation

I want better control over sending options:

  • I want to use a --partition-key when sending a single event.
  • I want to use a --batch-size when sending multiple events.

Change description

This PR splits the send command into two specialized commands:

  • send-event sends only one event, allowing using the provided --partition-key option.
  • send-batch sends all events in one of multiple batches using the provided --batch-size option.

Additional Notes

Reviewer checklist

  • PR address a single concern.
  • PR title and description are appropriately filled.
  • Changes will be merged in main.
  • Changes are covered by tests.
  • Logging is meaningful in case of troubleshooting.
  • Docs are updated (at least the README.md, if needed).
  • History is clean, commit messages are meaningful (see CONTRIBUTING.md) and are well-formatted.

This PR splits the `send` command into two specialized commands:

- `send-event` sends only one event, allowing using the provided
  `--partition-key` option.
- `send-batch` sends all events in one of multiple batches using
  the provided `--batch-size` option.
@zmoog zmoog self-assigned this Jun 4, 2024
@zmoog zmoog merged commit f98be65 into main Jun 4, 2024
@zmoog zmoog deleted the zmoog/split-send branch June 4, 2024 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant