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: fix string() panic if txn is nil #58

Merged
merged 2 commits into from
Sep 8, 2015

Conversation

siddontang
Copy link
Member

No description provided.

@c4pt0r
Copy link
Member

c4pt0r commented Sep 8, 2015

Add a test please.

@siddontang
Copy link
Member Author

PTAL

@qiuyesuifeng
Copy link
Member

LGTM

1 similar comment
@ngaut
Copy link
Member

ngaut commented Sep 8, 2015

LGTM

ngaut added a commit that referenced this pull request Sep 8, 2015
@ngaut ngaut merged commit 5dc5bfe into master Sep 8, 2015
@ngaut ngaut deleted the siddontang/fix-session-string-panic branch September 8, 2015 02:59
YuJuncen pushed a commit to YuJuncen/tidb that referenced this pull request Apr 23, 2021
)

* remove deprecated worker-pool-size option from configuration

* address comment
YuJuncen pushed a commit to YuJuncen/tidb that referenced this pull request Apr 23, 2021
* backup: run backup first and then checksum

Running backup and checksum concurrently slows down the total time

Signed-off-by: Neil Shen <[email protected]>

* restore: run restore first then checksum

Signed-off-by: Neil Shen <[email protected]>

* *: unify checksum computation

Signed-off-by: Neil Shen <[email protected]>

* backup: log total backup checksum time cost

Signed-off-by: Neil Shen <[email protected]>

* checksum: use low priority

Signed-off-by: Neil Shen <[email protected]>

* backup: remove duplicated range schema build function

Signed-off-by: Neil Shen <[email protected]>
okJiang pushed a commit to okJiang/tidb that referenced this pull request Oct 19, 2021
* add CsvNullValue Cli usage

* Update cmd/dumpling/main.go

Co-Authored-By: kennytm <[email protected]>

Co-authored-by: kennytm <[email protected]>
Defined2014 pushed a commit to Defined2014/tidb that referenced this pull request Apr 13, 2022
iosmanthus pushed a commit to iosmanthus/tidb that referenced this pull request Oct 31, 2022
* fix gcutil.GetGCSafePoint

Signed-off-by: David <[email protected]>
iosmanthus pushed a commit to iosmanthus/tidb that referenced this pull request May 11, 2023
* gcutil: fixed GetGCSafePoint (pingcap#58)

* fix gcutil.GetGCSafePoint

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

* make check

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

---------

Signed-off-by: David <[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.

4 participants