Daniël van Eeden

Results 172 issues of Daniël van Eeden

### Proposal Details We use `json.Valid()` in https://github.com/pingcap/tidb to implement the [`JSON_VALID()`](https://docs.pingcap.com/tidb/dev/json-functions/) SQL function and would like to return the position on which the validation failed to the user. https://github.com/pingcap/tidb/blob/0875abde25467897e4e81ae6c18c5e8792ca28a9/pkg/expression/builtin_json.go#L1019...

Proposal

I would like to see a notification if https://www.githubstatus.com/ is reporting any issues. Same for other things like GitLab when configured.

Would be nice if this could should Jira notifications etc as well. However I don't know if Jira and/or Jira Cloud have a suitable API.

enhancement

## Bug Report This is the same issue as https://github.com/pingcap/tidb-insight/issues/62 The code that has the problem: https://github.com/pingcap/diag/blob/514745da91ed13cd240a94eab52bfb97a6f97238/collector/sysinfo/process.go#L150 On FreeBSD this causes one direct and one indirect build failure. This was...

## Feature Request The experimental features listed on https://docs.pingcap.com/tidb/stable/experimental-features and elsewhere are not recommended for production use. A `tiup diag check...` should be able to detect those and give proper...

## Bug Report On CentOS 7 with the default terminal configuration the light-yellow on white as seen here is hard to read: ![image](https://user-images.githubusercontent.com/1272980/149325344-959246bf-3a9b-4e0c-a349-fd08270755f9.png) What version of Diag are you using...

## Feature Request Starting TiDB with the log format set to JSON allows for structured logging. ``` ./bin/tidb-server -config

### What problem does this PR solve? Issue Number: close #54162 Problem Summary: ### What changed and how does it work? ### Check List Tests - [ ] Unit test...

release-note
size/XXL
ok-to-test
lgtm

### What problem does this PR solve? Issue Number: ref #54207 Problem Summary: ### What changed and how does it work? Using `json.Valid()` is probably not the best solution here...

size/M
do-not-merge/work-in-progress
release-note-none
ok-to-test
do-not-merge/needs-tests-checked

### What problem does this PR solve? Issue Number: close #54250 Problem Summary: ### What changed and how does it work? ### Check List Tests - [x] Unit test -...

sig/planner
release-note
size/XXL
ok-to-test