mysql_random_data_load icon indicating copy to clipboard operation
mysql_random_data_load copied to clipboard

Published 20 hours ago • verified publisher icon Percona-Lab

Accept TiDB Global Index column

Open mjonss opened this issue 6 months ago • 4 comments

Added handling of the extra column in TiDB 8.5 for Global Index during

SHOW INDEXES FROM `%s`.`%s`

mjonss avatar May 22 '25 17:05 mjonss

@mjonss Thanks for your contribution. I am currently analyzing the rewrite branch and I am interested about your change: Ultimately, what does having this extra index handled by mysql_random_data_load enable for you ?

Indexes are stored based on the "table" struct, but there's not so much usage of it. I'd like to understand better how this impacts TIDB usage, I would appreciate if you can help me understand

ylacancellera avatar Jun 06 '25 12:06 ylacancellera

@mjonss Thanks for your contribution. I am currently analyzing the rewrite branch and I am interested about your change: Ultimately, what does having this extra index handled by mysql_random_data_load enable for you ?

Indexes are stored based on the "table" struct, but there's not so much usage of it. I'd like to understand better how this impacts TIDB usage, I would appreciate if you can help me understand

@ylacancellera "having" this index is not useful, but allowing mysql_random_data_load to work with the extended output from TiDB's SHOW INDEXES allows it to also be used for TiDB :)

What I am trying to say is that all that is needed is to allow more columns in the output from SHOW INDEXES, they can be ignored, but the current implementation seems to need to parse them from the output, through row.Scan(fields...).

mjonss avatar Jun 10 '25 15:06 mjonss

@mjonss so that you know: there should be in the near future (probably few weeks) a fork of this project in a new repo. This is to merge this branch and to add pg support amount other improvments. It is not using SHOW INDEXES anymore, so I would imagine it will work for tidb as you expect, though I did not test.

Thanks for your contribution

ylacancellera avatar Jun 17 '25 10:06 ylacancellera

Please feel free to close this when the new repo is launched. I also have some ideas generate data according to gathered statistics from TiDB or MySQL or MariaDB. So I'm interested in the new repo :) This would help troubleshooting optimizer issues, test plans and validate ideas.

mjonss avatar Jun 17 '25 20:06 mjonss