-
Notifications
You must be signed in to change notification settings - Fork 3k
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
enhance: support null in c data_datacodec and load null value #32183
enhance: support null in c data_datacodec and load null value #32183
Conversation
@smellthemoon E2e jenkins job failed, comment |
@smellthemoon ut workflow job failed, comment |
6b57719
to
6f91aab
Compare
@smellthemoon ut workflow job failed, comment |
@smellthemoon E2e jenkins job failed, comment |
6f91aab
to
0022575
Compare
@smellthemoon E2e jenkins job failed, comment |
@smellthemoon ut workflow job failed, comment |
0022575
to
2adb77a
Compare
@smellthemoon ut workflow job failed, comment |
@smellthemoon E2e jenkins job failed, comment |
eb18906
to
fad8f1d
Compare
@smellthemoon E2e jenkins job failed, comment |
1 similar comment
@smellthemoon E2e jenkins job failed, comment |
@smellthemoon ut workflow job failed, comment |
fad8f1d
to
5c1bce2
Compare
@smellthemoon E2e jenkins job failed, comment |
@smellthemoon ut workflow job failed, comment |
5c1bce2
to
d817e0e
Compare
@smellthemoon E2e jenkins job failed, comment |
@smellthemoon ut workflow job failed, comment |
d817e0e
to
d1ae4f5
Compare
@smellthemoon E2e jenkins job failed, comment |
@smellthemoon ut workflow job failed, comment |
ab9f49e
to
ea91d22
Compare
ea91d22
to
407a0e2
Compare
@smellthemoon E2e jenkins job failed, comment |
4ab36e2
to
0167033
Compare
@smellthemoon E2e jenkins job failed, comment |
0167033
to
65657cd
Compare
Signed-off-by: lixinguo <[email protected]>
Signed-off-by: lixinguo <[email protected]>
Signed-off-by: lixinguo <[email protected]>
Signed-off-by: lixinguo <[email protected]>
Signed-off-by: lixinguo <[email protected]>
65657cd
to
a4835f6
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #32183 +/- ##
==========================================
- Coverage 80.96% 80.92% -0.04%
==========================================
Files 1185 1180 -5
Lines 142715 143366 +651
==========================================
+ Hits 115544 116018 +474
- Misses 22760 22917 +157
- Partials 4411 4431 +20
|
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: congqixia, smellthemoon The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
will store valid_data(use uint8_t type to save memory) in fieldData.
binlog reader read and write data into column(sealed segment), insertRecord(growing segment). In sealed segment, store valid_data directly. In growing segment, considering prior implementation and easy code reading, it covert uint8_t to fbvector, which may optimize in future.
parse valid_data in search/query.
[Feature]: Support Null and default value #31728