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

When registering as a slave we should be using the host's hostname, not the server's hostname. #34

Closed
gdey opened this issue Jan 22, 2016 · 0 comments

Comments

@gdey
Copy link
Contributor

gdey commented Jan 22, 2016

We have a bug in writeRegisterSlaveCommand (https://github.com/siddontang/go-mysql/blob/master/replication/binlogsyncer.go#L414-L417) where we are using the host name of the server we are connecting to instead of the host we are on.

This makes is confusing when looking at the show slave status output on the master.

We should just use os.Hostname() here.

gdey added a commit to gdey/go-mysql that referenced this issue Jan 22, 2016
When registering as a slave we are using the wrong hostname. It should
be our hostname, not the hostname name of the server we are connecting
to.
gdey added a commit to gdey/go-mysql that referenced this issue Jan 22, 2016
When registering as a slave we are using the wrong hostname. It should
be our hostname, not the hostname name of the server we are connecting
to.

Fixed issue with using the wrong variable.
siddontang added a commit that referenced this issue Jan 26, 2016
This fixes issue #34. Using the wrong hostname.
@gdey gdey closed this as completed Jan 26, 2016
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

No branches or pull requests

1 participant