pingcap / pingcap/tidb

Same query results are inconsistent on TiKV and TiFlash when involving `RIGHT` function.

Open
#57,854 2 comments 0 reactions 0 assignees View on GitHub
severity/moderate sig/execution type/bug
Dominant language
Go
Stars
40.5k
Forks
6.2k
PR merge metrics
PR metrics pending

Description

## Bug Report

Please answer these questions before submitting your issue. Thanks!

### 1. Minimal reproduce step (Required)
prepare data
```sql
drop table if exists t0;
CREATE TABLE t0(c0 BIGINT PRIMARY KEY);
INSERT INTO t0 VALUES (1),(2),(3),(4),(5),(6),(7);
INSERT IGNORE INTO t0(c0) VALUES (-1);
ALTER TABLE t0 SET TIFLASH REPLICA 1;

```
execute query
#### May require multiple executions to reproduce.
```sql
SELECT /*+ read_from_storage(tikv[t0]) */ t0.c0 FROM t0 WHERE RIGHT(t0.c0, ((t0.c0)^('1')));
SELECT /*+ read_from_storage(tiflash[t0]) */ t0.c0 FROM t0 WHERE RIGHT(t0.c0, ((t0.c0)^('1')));
```

### 2. What did you expect to see? (Required)
The results of the two queries are equal.

### 3. What did you see instead (Required)
Query1:
![Image](https://github.com/user-attachments/assets/32dc32b9-80c8-4779-90f9-058c90f0c190)
Query2:
![Image](https://github.com/user-attachments/assets/b6027f10-720c-47dc-9a3f-9974d577b506)

### 4. What is your TiDB version? (Required)

Release Version: v8.4.0
Edition: Community
Git Commit Hash: 1a9f0fa25580907d3266dc3edf000ba24379a600
Git Branch: HEAD
UTC Build Time: 2024-11-07 15:18:43
GoVersion: go1.23.2
Race Enabled: false
Check Table Before Drop: false
Store: tikv

Contributor guide

Open the contributing guide

Research direction

Run the provided setup and execute both SELECT statements, using the TiKV and TiFlash storage hints, then compare their results. Trace the RIGHT expression's evaluation along the two storage paths and verify the fix by confirming both queries return equal results on the reproduction data.

Written by the indexing model from the issue text.

Assessment

Tech stack
go, sql
Domain
databases, distributed-systems
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.