[MDEV-22265] Connect string character limit too small for full 64 character InnoDB table-name limit when using ad-hoc Spider server definitions. Created: 2020-04-16 Updated: 2021-12-16 Resolved: 2021-05-01 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | Storage Engine - Spider |
| Affects Version/s: | 10.4.10 |
| Fix Version/s: | 10.4.19, 10.5.10 |
| Type: | Bug | Priority: | Critical |
| Reporter: | Juan | Assignee: | Alexey Botchkov |
| Resolution: | Fixed | Votes: | 1 |
| Labels: | None | ||
| Environment: |
CentOS 7 |
||
| Attachments: |
|
| Description |
|
When creating spider tables with ad-hoc server parameters (ie. not a reference to mysql.servers but host, port, database, user, password in the table definition) the spider table creation process throws "ERROR 12502 (HY000): The connect info for table is too long" before the 64-character tabe-name length limit is reached, causing some tables on source servers to be inaccessible as spider connect tables to user who cannot declare spider servers. The length limit of the connect string is a little bit complicated to calculate exactly because implicit table names are expanded by spider and hostnames can also be resolved. The limit appears to be 149 characters fully resolved. For example:
where
and
|
| Comments |
| Comment by Kentoku Shiba (Inactive) [ 2020-12-18 ] |
|
I couldn't reproduce this issue. Is there any test environment that someone already reproduced? |
| Comment by Juan [ 2020-12-24 ] |
|
Hi Kentoku - I just set up some instances for you on EC2 to walk you through the reproduction, but could not get it to happen the same way as 8 months ago either on the latest 10.5 or on 10.4.12 in CentOS 7. I went through notes and found the original instances I used for that repro. I'll get those from backups and get you a more detailed walk-through in the coming days. Happy holidays! |
| Comment by Alexey Botchkov [ 2021-04-23 ] |
|
proposed fix. https://github.com/MariaDB/server/commit/162761fb43747b1690ac396a7809afd1c4eee5c2 |
| Comment by Sergei Golubchik [ 2021-04-26 ] |
|
ok to push |