-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Refactor PMA_setGlobalVariablesForEngine #12703
Comments
How do I assign this issue to me? I will give it a try! |
You can give it a try :-). AFAIK nobody is working on this. |
I was looking through the code and I think that this can be simplified a lot. There is no need to have several boolean variables in the Table class just to tell if it is a kind of storage engine or not. For instance, I will add a variable in the Table class that will hold the string value of the select storage engine (ie "InnoDb") and instead of
I can simply have: |
I think motivation for this was to avoid doing string comparison too many times as this is something what is used quite often. But maybe in the end this is just overoptimized and string comparison would work fine. In either case this is used quite often, so I'd prefer some nicer interface to this, for example having |
Currently
PMA_setGlobalVariablesForEngine
returns long list of variables which are then passed around the code. This is really bad design patter, it should be made more transparent to the users. One of possibilities is to move the code to the Table class and use it from there (with caching).See also #12567
The text was updated successfully, but these errors were encountered: