-
Notifications
You must be signed in to change notification settings - Fork 687
Parse gtid events #85
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
Parse gtid events #85
Conversation
6da3bac
to
e25628c
Compare
|
||
@property | ||
def gtid(self): | ||
gtid = "%s%s%s%s-%s%s-%s%s-%s%s-%s%s%s%s%s%s" %\ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there any way I can understand what this line is doing? :D
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:))
I'm trying to restore the gtid format
see http://dev.mysql.com/doc/refman/5.6/en/replication-gtids-concepts.html
Couldn't find a better implementation of it :/
e25628c
to
5e96c89
Compare
def gtid(self): | ||
"""GTID = source_id:transaction_id | ||
Eg: 3E11FA47-71CA-11E1-9E33-C80AA9429562:23""" | ||
gtid = "%s%s%s%s-%s%s-%s%s-%s%s-%s%s%s%s%s%s" %\ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bjoernhaeuser Is it more clear this way ?
What do you think about adding the link directly into the source code? I would merge it afterwards. Thanks! Feel free to add yourself to the contributors list :) |
Nice contribution @baloo , i agree with @bjoernhaeuser we can add the link in the source it will help us in the futur. |
5e96c89
to
465605f
Compare
done |
BTW, i'm currently trying to implement the MASTER_AUTO_POSITION feature with gtids. PR to come. |
Thanks it's merged |
No description provided.