Skip to content

Conversation

SzymonPobiega
Copy link
Member

@SzymonPobiega SzymonPobiega commented Feb 10, 2025

Adds support to new topology to the command line tool. New usage:

  • Create infrastructure for a new endpoint using topic-per-event-type topology
endpoint create MyEndpoint
  • Create infrastructure for a new endpoint using legacy migration topology (includes creating the bundle topic and a subscription)
migration endpoint create MyEndpoint
  • Subscribe / unsubscribe using the topic-per-event-type topology
endpoint subscribe MyEndpoint MyEvent
endpoint unsubscribe MyEndpoint MyEvent
  • Subscribe / unsubscribe using the legacy migration topology
migration endpoint subscribe MyEndpoint MyEvent
migration endpoint unsubscribe MyEndpoint MyEvent
  • Subscribe / unsubscribe using the topic-per-event-type topology in a migration endpoint
migration endpoint subscribe-migrated MyEndpoint MyEvent
migration endpoint unsubscribe-migrated MyEndpoint MyEvent

@SzymonPobiega
Copy link
Member Author

@danielmarbach this is now in the form that I think is ready for a review.

@danielmarbach
Copy link
Contributor

I'm wondering if it wouldn't be better to have a more drastic approach with the tooling by moving everything migration related underneath a command.
So basically, you can continue using endpoint create, but that does it in the topic-per-event type way and if you want to access the "old commands" you have to do "migrate create" or migrate subscribe etc. which that it would also follow more logically what we have in the transport, and it is better optimized for removal because you can delete an entire command tree when migration is no longer supported

@SzymonPobiega SzymonPobiega merged commit 058a17b into master Feb 12, 2025
3 checks passed
@SzymonPobiega SzymonPobiega deleted the command-line branch February 12, 2025 10:09
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.

2 participants