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

Support the ability to deploy a Kafka proxy #407

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

savnadya
Copy link
Collaborator

No description provided.

@@ -60,4 +61,8 @@ const (

MasterCachesRPCPort = 9018
MasterCachesMonitoringPort = 10018

KafkaProxyMonitoringPort = 10020
Copy link
Contributor

Choose a reason for hiding this comment

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

Two requests here.

  1. Monitoring port should be equal to YT RPC port + 1000.
  2. I am almost sure that 9020, 9021 and 9023 are occupied by some other services (maybe through Y internal configuration system). AFAIR, 9020 is a standard port for the discovery servers, therefore there will be a clash of monitoring ports. I'd prefer not to introduce collisions or inconsistencies with the internal Y conventions here, so I suggest choosing higher two ports (rpc and kafka), ideally the same as you already have inside.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants