[regexp lib] TiKV's regexp is inconsistent with that of Tidb

[knull-cn]

Bug Report

What version of TiKV are you using?

v6.5.x, v7.1.x

What operating system and CPU are you using?

1tikv (72c/385g)

Steps to reproduce

create table TT (str varchar(1000));
insert into TT (str) values('^/home/ap/ncbs/file/output/fa/xbrl/\d{4}([0]{1}[1-9]{1}|[1]{1}[012]{1})([0]{1}[1-9]{1}|[12]{1}\d{1}|[3]{1}[01]{1})/IVSR_HOLD_FNDLOT_PERCN_\d{2}\_\d{8}\.txt$');
select 1 from TT where 'abc' regexp str;

What did you expect?

no error

What did happened?

[knull-cn]

same query run on tidb v6.1.1 was correct
because the regexp was run on tidb-server at v6.1.1；and pushdown to tikv v6.5.x
means , regexp was different between on golang and rust

[seiya-annie]

/found gs

[jebter]

/label affects-6.5
/label affects-7.1

[gengliqi]

The escape \ for \_ is useless.
So the reason is as rust-lang/regex#497 said,

Inconsequential escapes are not allowed.

If you delete this \, no error will be reported.

Most of the behavior between the regexp library of Go and Rust is the same, but a few behaviors like this one are different. Only when we unify the three compute engines(tidb,tikv,tiflash) can all behaviors be truly consistent.

[zanmato1984]

This seems a duplicate of pingcap/tidb#48360

Let's close this one and track the issue in the other.

[zanmato1984]

/close

[ti-chi-bot]

@zanmato1984: You can't close an active issue/PR unless you authored it or you are a collaborator.

In response to this:

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[jebter]

/close

[ti-chi-bot]

@jebter: Closing this issue.

In response to this:

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[kennedy8312]

/type regression

[ti-chi-bot]

@kennedy8312: The label(s) type/regression cannot be applied, because the repository doesn't have them.

In response to this:

/type regression

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the ti-community-infra/tichi repository.