pingcap / pingcap/tidb

attach2task should not change anything about the wrapped / shallow-refed / shared p inside

Open
#61,345 0 comments 0 reactions 0 assignees View on GitHub
planner/cascades sig/planner type/enhancement
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)

![Image](https://github.com/user-attachments/assets/cb2e22ae-4b42-4188-85b1-779857be7d49)
a simple example about change the common-shared p inside a task from children.

![Image](https://github.com/user-attachments/assets/155bebd1-4808-4d46-b38a-a72f540d2616)

### 2. What did you expect to see? (Required)
attach2Task should NOT change anything about wrapped/shallowed/shared p inside.

### 3. What did you see instead (Required)
for volcano model, since each logical op only have exact one logical op parent, for this logical op's all enumeration of physical plans, we enumerate them according the possible props listed, but sometimes it is still possible to happen what mentions above.

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

master

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.