-
Notifications
You must be signed in to change notification settings - Fork 101
lightning: fix parquet parser for decimal type (#1272) #1277
lightning: fix parquet parser for decimal type (#1272) #1277
Conversation
[REVIEW NOTIFICATION] This pull request has been approved by:
To complete the pull request process, please ask the reviewers in the list to review by filling The full list of commands accepted by this bot can be found here. Reviewer can indicate their review by submitting an approval review. |
/merge |
This pull request has been accepted and is ready to merge. Commit hash: 5015cff
|
/run-all-tests |
/run-integration-tests |
/run-all-tests |
…-pick-1272-to-release-5.0
/run-all-tests |
/run-integration-test /merge |
This pull request has been accepted and is ready to merge. Commit hash: a947215
|
/run-integration-test |
looks like cdc has some breaking change in https://github.com/pingcap/ticdc/pull/2070 |
It is not a breaking change, but there exists a bug in TiCDC. Can add |
@amyangfei is this flag available only on 5.0+? |
…-pick-1272-to-release-5.0
/merge |
This pull request has been accepted and is ready to merge. Commit hash: f092cd1
|
ticdc 4.0 will also cherry-pick this change https://github.com/pingcap/ticdc/pull/2069 @kennytm |
/run-all-tests |
This is an automated cherry-pick of #1272
What problem does this PR solve?
Fix parse parquet file when the corresponding type is decimal with
FIXED_LEN_BYTE_ARRAY
orBINARY
What is changed and how it works?
Check List
Tests
Code changes
Side effects
Related changes
Release note