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

mysql: binlog_streamer unexpected panic #4270

Merged
merged 1 commit into from
Oct 17, 2018
Merged

Conversation

sougou
Copy link
Contributor

@sougou sougou commented Oct 12, 2018

The binlog_streamer experienced a panic while processing
JSON data. It seems that there are situations where things
are empty. This is defensive code against such an occurence.

Signed-off-by: Sugu Sougoumarane ssougou@gmail.com

@sougou
Copy link
Contributor Author

sougou commented Oct 13, 2018

@hmcgonig to review

The binlog_streamer experienced a panic while processing
JSON data. It seems that there are situations where things
are empty. This is defensive code against such an occurence.

Signed-off-by: Sugu Sougoumarane <ssougou@gmail.com>
@hmcgonig
Copy link
Contributor

This sounds reasonable to me, thanks for looking into this!

@sougou sougou merged commit b019958 into vitessio:master Oct 17, 2018
@sougou sougou deleted the ss-json branch October 30, 2018 02:56
systay pushed a commit that referenced this pull request Jul 22, 2024
…keys (#4270)

* cherry pick of 15081

* simplify the mycnf test

Signed-off-by: Harshit Gangal <harshit@planetscale.com>

---------

Signed-off-by: Harshit Gangal <harshit@planetscale.com>
Co-authored-by: Harshit Gangal <harshit@planetscale.com>
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