-
Notifications
You must be signed in to change notification settings - Fork 13
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
Mariadb4.x galera ps API v2 #160
base: master
Are you sure you want to change the base?
Conversation
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.
In my opinion this looks good. I had only a very minor comments so you can if so choose ignore them and please send this to review for Teemu Ollakka.
src/wsrep_provider_v26.cpp
Outdated
wsrep::ps_service* ps_service) | ||
{ | ||
assert(ps_service); | ||
if (not wsrep::ps_service_v1_probe(dlh)) |
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 have not seen this before, does this compile in our old compilers?
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.
fixed
src/wsrep_provider_v26.cpp
Outdated
|
||
static void deinit_ps_service(void* dlh) | ||
{ | ||
// assert(not wsrep::ps_service_v1_probe(dlh)); |
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 remove commented code lines, why this would not hold here?
src/ps_service_v1.cpp
Outdated
int wsrep::ps_service_v1_init(void* dlh, | ||
wsrep::ps_service* ps_service) | ||
{ | ||
if (not (dlh && ps_service)) return EINVAL; |
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.
Why not instead of ! ?
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.
fixed
This commit adds support for Performanche Schema tables for Galera using the new Pefrormance Schema (ps) service.
2ae0399
to
1f2a908
Compare
This is a prototype of a new implementation of the interface to support performance schema tables in MariaDB, which does not require changes to the wsrep-API as it uses the wsrep service architecture, similar to the service for TLS.