-
Notifications
You must be signed in to change notification settings - Fork 976
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
row_event.go: fix the set bytes value in revert order #319
Conversation
if no document about it, we should test |
should add a test @july2993 |
Similar bug occurs when handle |
will fix it later and add test |
@siddontang PTAL |
@@ -523,6 +523,71 @@ func (_ *testDecodeSuite) TestParseJsonDecimal(c *C) { | |||
} | |||
} | |||
|
|||
func (_ *testDecodeSuite) TestEnum(c *C) { |
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.
Maybe we should add test that covers case length=2
in row_event.go
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 make a mistake, should add more than 256 enum num but not more than 8 to test case length=2
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.
please add a new test later @july2993
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.
@siddontang add here #320
fix pingcap/tidb-tools#70
according to the binlog, we should handle it in little end
i have test more when the set num is 18 (need 3 bytes), but i can't find a doc from mysql about this