Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tidb: Split datasource name to uri and dbName #96

Merged
merged 2 commits into from
Sep 10, 2015

Conversation

shenli
Copy link
Member

@shenli shenli commented Sep 10, 2015

The DBName may be different from data path name.

The DBName may be different from data path name.
name = name[len("file://"):]
case strings.HasPrefix(name, "memory://"):
name = name[len("memory://"):]
case strings.HasPrefix(uri, "file://"):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does any engine use file:// as the prefix ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The switch statement is useless now. I will remove it.

@shenli
Copy link
Member Author

shenli commented Sep 10, 2015

PTAL

@siddontang
Copy link
Member

LGTM

shenli added a commit that referenced this pull request Sep 10, 2015
tidb: Split datasource name to uri and dbName
@shenli shenli merged commit aa3b38a into master Sep 10, 2015
@shenli shenli deleted the shenli/split-uri-dbname branch September 10, 2015 09:48
YuJuncen pushed a commit to YuJuncen/tidb that referenced this pull request Apr 23, 2021
1. Ensure FileCheckpointsDB is mutex-protected.
   This prevents the "concurrent map iteration and map write" error.
2. Split the DDLs creating a MySQLCheckpointsDB into 3 separate statements
3. Reduced the path length of chunks_v3 in the PRIMARY KEY to 500 chars
   to avoid the 3072 key-length limitation.
YuJuncen pushed a commit to YuJuncen/tidb that referenced this pull request Apr 23, 2021
xhebox pushed a commit to xhebox/tidb that referenced this pull request Sep 28, 2021
xhebox pushed a commit to xhebox/tidb that referenced this pull request Oct 8, 2021
mittalrishabh pushed a commit to mittalrishabh/tidb that referenced this pull request May 30, 2024
…#53335) (pingcap#96)

* *: add rpc error stats and refine log for diagnose (pingcap#53335)

* sessionctx: add sysvar to control exp feats of replica selector

Signed-off-by: zyguan <[email protected]>

* *: add rpc error stats and refine log (pingcap#52810)

Signed-off-by: crazycs520 <[email protected]>

* store: refine coprocessor error log (pingcap#52729)

Signed-off-by: crazycs520 <[email protected]>

* *: improve log about stale-read query (pingcap#52494)

Signed-off-by: crazycs520 <[email protected]>

* *: refine tikv/client-go log to print context information(such as conn id) as much as possible (pingcap#45559)

Signed-off-by: crazycs520 <[email protected]>

* add leader peer id log

Signed-off-by: crazycs520 <[email protected]>

* update log

Signed-off-by: crazycs520 <[email protected]>

* executor: fix issue that some query execution stats was omitted when execution was interrupted (pingcap#51787)

close pingcap#51660

Signed-off-by: crazycs520 <[email protected]>

* update go.mod

Signed-off-by: crazycs520 <[email protected]>

---------

Signed-off-by: zyguan <[email protected]>
Signed-off-by: crazycs520 <[email protected]>
Co-authored-by: zyguan <[email protected]>

* Update go.sum

* Update go.mod

---------

Signed-off-by: zyguan <[email protected]>
Signed-off-by: crazycs520 <[email protected]>
Co-authored-by: crazycs <[email protected]>
Co-authored-by: zyguan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants