[MDEV-30576] Include spider error codes in extra/perror Created: 2023-02-06 Updated: 2023-05-27 |
|
| Status: | Open |
| Project: | MariaDB Server |
| Component/s: | Storage Engine - Spider |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major |
| Reporter: | Yuchen Pei | Assignee: | Yuchen Pei |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||||||||||
| Description |
|
Requested by Roel.
Other observed codes: 12701, 12501 |
| Comments |
| Comment by Yuchen Pei [ 2023-03-28 ] |
|
I guess the proper way of fixing this should be moving spider errors from spd_err.h to errmsg-utf8.txt, which will also make it automatically localised. |
| Comment by Sergei Golubchik [ 2023-05-23 ] |
|
no, the proper way would be to extend plugin API to allow plugins to have their own translated error messages. |
| Comment by Yuchen Pei [ 2023-05-24 ] |
|
serg I assume this rule applies to dynamic plugins only, i.e. those not in mysql_mandatory_plugins and mysql_optional_plugins, right? I mean there are plenty of things in the server code helping out builtin plugins like InnoDB, perfschema, aria etc. |
| Comment by Sergei Golubchik [ 2023-05-24 ] |
|
it's a general rule with exception. there are few plugins who are more integrated into the server, unfortunately. Spider could be, technically, compiled statically too, if you remove MODULE_ONLY from its CMakeLists.txt |
| Comment by Yuchen Pei [ 2023-05-25 ] |
|
> Spider could be, technically, compiled statically too, if you remove MODULE_ONLY from its CMakeLists.txt or do PLUGIN_SPIDER:STRING=STATIC in CMakeCache.txt |
| Comment by Roel Van de Paar [ 2023-05-27 ] |
|
Another example here. |