Skip to content

add Kafka setup to Pub/Sub example#33

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

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

Conversation

@tomkralidis

Copy link
Copy Markdown
Member

Adding based on geopython/pygeoapi#2258

@doublebyte1 doublebyte1 left a comment

Copy link
Copy Markdown
Contributor

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
Copy Markdown
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.

@doublebyte1 doublebyte1 left a comment

Copy link
Copy Markdown
Contributor

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