matrixorigin / matrixone

Hyperconverged cloud-edge native database
https://docs.matrixorigin.cn/en
Apache License 2.0
1.76k stars 273 forks source link

[Bug]: build_ddl.go, line 3286, indexs is never initialized therefore the loop is a no op. #17576

Closed fengttt closed 6 days ago

fengttt commented 1 month ago

Is there an existing issue for the same bug?

Branch Name

main

Commit ID

04eb3e8001220c65645fc9db971d76367e9ad2cc

Other Environment Information

- Hardware parameters:
- OS type:
- Others:

Actual Behavior

It is obviously a bug, from code inspection. In fact, golang gives a warning.

The loop is a noop, which is more or less fine. I am worried that we are missing some case that we should have build the indexs before the loop.

Expected Behavior

No response

Steps to Reproduce

Read the code

Additional information

No response

qingxinhome commented 1 month ago

The issue has been resolved

aressu1985 commented 1 month ago

这个问题田博确认一下