Open YangKeao opened 2 weeks ago
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please assign windtalker for approval. For more information see the Code Review Process.
The full list of commands accepted by this bot can be found here.
/hold
I remembered another issue: https://github.com/pingcap/tidb/issues/50370. I need to check whether this PR will bring some potential correctness issue later :thinking: .
Attention: Patch coverage is 95.23810%
with 1 lines
in your changes are missing coverage. Please review.
Project coverage is 74.5644%. Comparing base (
1c4a9c6
) to head (6abc68d
). Report is 46 commits behind head on master.
/retest
@YangKeao: The following test failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
fast_test_tiprow | 6abc68dae76fe0824c3a089fe32d911a56d1b5d0 | link | true | /test fast_test_tiprow |
Full PR test history. Your PR dashboard.
What problem does this PR solve?
Issue Number: close #53243, close #53352
Problem Summary:
Two problem:
DATE/DATETIME/TIMESTAMP array
.fsp
when converting JSON toDATE/DATETIME/TIMESTAMP
.What changed and how does it work?
DATE/DATETIME/TIMESTAMP
when we are converting them toDATE/DATETIME/TIMESTAMP
array.GetTime
which uses theDefaultFsp
, and implement a new method onBinaryJSON
to returntypes.Time
with the providedFsp
.Check List
Tests
Side effects
Documentation
Release note
Please refer to Release Notes Language Style Guide to write a quality release note.