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

Removed SR store implementation detail from wsrep-lib #199

Open
wants to merge 1 commit into
base: 10.7-sr-speedup
Choose a base branch
from

Conversation

temeo
Copy link
Contributor

@temeo temeo commented Oct 7, 2022

The following was moved to application side implementation:

  • Removed sr_store from streaming context.
  • Removed sr_state from transaction.
  • Removed get_binlog_cache() from client_service interface.

Other:

  • Add SR applier reference to append_fragment_and_commit() to
    make it available for application.
  • Add separate interface call to rollback SR transactions
    on disconnect. Rolling back SR transactions due to rollback
    fragment and rolling back SR transactions due to disconnect have
    different behaviors. Have separate calls for these different
    cases for clarity.
  • Remove non-const transaction accessor, not needed anymore because
    SR state has been moved to application side.
  • Remove unneeded set_fragments_from_table().

The following was moved to application side implementation:

- Removed sr_store from streaming context.
- Removed sr_state from transaction.
- Removed get_binlog_cache() from client_service interface.

Other:
- Add SR applier reference to append_fragment_and_commit() to
  make it available for application.
- Add separate interface call to rollback SR transactions
  on disconnect. Rolling back SR transactions due to rollback
  fragment and rolling back SR transactions due to disconnect have
  different behaviors. Have separate calls for these different
  cases for clarity.
- Remove non-const transaction accessor, not needed anymore because
  SR state has been moved to application side.
- Remove unneeded set_fragments_from_table().
@temeo temeo requested a review from plampio October 7, 2022 09:19
Copy link

@plampio plampio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

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