Support for Tagged GTIDs in Previous-GTIDs events#8
Conversation
|
@EeshwarPrasath Hi, thanks for the PR. IT looks good yet, I'd like to have it tested to. Would it be possible to add an integration test that will enable tagged transaction and run it against MySQL 8.4? We'd then add MySQL 8.4 test workflow to the GitHub repository. It would bprobably need few more steps
|
|
Hi @jpechane, Thank you for reviewing the PR and providing detailed instructions. I’m happy to work on the integration test as requested. However, adding support for MySQL 8.4 to onetimeserver is taking longer than anticipated due to some complexities. If anyone else from the community has insights or suggestions for adding MySQL 8.4 support to onetimeserver, I’d appreciate the input. Thanks for your understanding. |
|
@EeshwarPrasath Thanks! BTW, for our purposes I believe it is sufficient to do support for Linux only. In future i can imagine we'd replace this approach with |
|
❌ Developer Certificate of Origin (DCO) check failed. Hi @EeshwarPrasath, please sign off all commits with: If pull request commits are not signed off, the pull request cannot be merged. For more information about why this is required, please see our blog about contribution requirement changes. |
Signed-off-by: Eeshwar Prasath S A <eeshwarprasath@outlook.in>
|
Hi @Naros, I have signed off the commit. Kindly check. |
|
Looks good, thanks @EeshwarPrasath 🚀 |
|
@EeshwarPrasath Applied, thanks! |
Previously, we were unable to parse Tagged GTIDs in Previous-GTIDs events since the tags and its related data were embedded differently inside the Previous-GTIDs events whenever the tags were enabled, and trying to parse them always throwed exceptions.
This PR solves the issue mentioned here: osheroff#158 (comment)