pingcap / pingcap/tidb

extstore should use a dedicated storage URI sysvar

Open
#67,214 0 comments 0 reactions 0 assignees View on GitHub
sig/planner type/enhancement
Dominant language
Go
Stars
40.5k
Forks
6.2k
PR merge metrics
PR metrics pending

Description

### Problem
`extstore` currently reads `tidb_cloud_storage_uri`, which is also used by DDL add index and import related flows. This couples extstore storage selection with unrelated features and makes it hard to configure extstore independently.

### Proposed change
Add a dedicated global system variable, `tidb_external_storage_uri`, for extstore.

### Expected behavior
- `extstore` reads `tidb_external_storage_uri` instead of `tidb_cloud_storage_uri`
- the new variable keeps the same URI validation and redaction behavior as `tidb_cloud_storage_uri`
- `tidb_cloud_storage_uri` remains available for its existing users

### Affected area
- extstore
- sysvars / session variable plumbing
- redaction / invisibility handling

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.