Skip to content

add Kafka setup to Pub/Sub example#33

Merged
doublebyte1 merged 3 commits intomainfrom
pubsub-kafka
Feb 27, 2026
Merged

add Kafka setup to Pub/Sub example#33
doublebyte1 merged 3 commits intomainfrom
pubsub-kafka

Conversation

@tomkralidis
Copy link
Member

Adding based on geopython/pygeoapi#2258

Copy link
Contributor

@doublebyte1 doublebyte1 left a comment

Choose a reason for hiding this comment

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

Thanks a lot for adding this! 🥇 I was going to suggest it, when you did the PR on pygeoapi.

Would you mind:

  • Moving the pubsub folder inside the top-level docker folder? mv pubsub ../docker
  • Renaming the docker folder to something more meaningful like config or pygeoapi? mv pubsub/docker pubsub/pygeoapi

@tomkralidis
Copy link
Member Author

Thanks a lot for adding this! 🥇 I was going to suggest it, when you did the PR on pygeoapi.

Would you mind:

  • Moving the pubsub folder inside the top-level docker folder? mv pubsub ../docker
  • Renaming the docker folder to something more meaningful like config or pygeoapi? mv pubsub/docker pubsub/pygeoapi

@doublebyte1 PR fixed / updated.

Copy link
Contributor

@doublebyte1 doublebyte1 left a comment

Choose a reason for hiding this comment

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

Thanks a lot, I think this example is incredibly valuable.

@doublebyte1 doublebyte1 merged commit 02affb2 into main Feb 27, 2026
@tomkralidis tomkralidis deleted the pubsub-kafka branch February 27, 2026 15:23
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