Skip to content

[Bug] fix wrong null flag#10883

Merged
dataroaring merged 3 commits intoapache:masterfrom
Gabriel39:fix_conpaction
Jul 16, 2022
Merged

[Bug] fix wrong null flag#10883
dataroaring merged 3 commits intoapache:masterfrom
Gabriel39:fix_conpaction

Conversation

@Gabriel39
Copy link
Contributor

Proposed changes

Issue Number: close #xxx

Problem Summary:

Describe the overview of changes.

Checklist(Required)

  1. Does it affect the original behavior: (Yes/No/I Don't know)
  2. Has unit tests been added: (Yes/No/No Need)
  3. Has document been added or modified: (Yes/No/No Need)
  4. Does it need to update dependencies: (Yes/No)
  5. Are there any changes that cannot be rolled back: (Yes/No)

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

@Gabriel39 Gabriel39 force-pushed the fix_conpaction branch 2 times, most recently from 2e88857 to 762fb56 Compare July 15, 2022 09:21
@cambyzju
Copy link
Contributor

Why not adjust null_map pos inside set_source_column?

yiguolei
yiguolei previously approved these changes Jul 15, 2022
Copy link
Contributor

@yiguolei yiguolei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Jul 15, 2022
@github-actions
Copy link
Contributor

PR approved by at least one committer and no changes requested.

@github-actions
Copy link
Contributor

PR approved by anyone and no changes requested.

@Gabriel39
Copy link
Contributor Author

set_source_column

@cambyzju If we do so, inside get_data_at, offset will be never applicate for nullmap any more so we have to add a different offset for column data and null map. I think it's a bad implement so I just did this.

@dataroaring dataroaring merged commit 75ca21d into apache:master Jul 16, 2022
eldenmoon pushed a commit to eldenmoon/incubator-doris that referenced this pull request Aug 1, 2022
yiguolei pushed a commit that referenced this pull request Apr 3, 2023
The offset of _nullmap and _value are inconsistent in OlapDataConvertor, so the obtained null flag is incorrect when calling get_ data_ at function. When the key column or sequence column has null values, the encoding of the short key index or primary key index may be wrong.
This was introduced by #10883 #10925.
zhannngchen pushed a commit to zhannngchen/incubator-doris that referenced this pull request Apr 3, 2023
The offset of _nullmap and _value are inconsistent in OlapDataConvertor, so the obtained null flag is incorrect when calling get_ data_ at function. When the key column or sequence column has null values, the encoding of the short key index or primary key index may be wrong.
This was introduced by apache#10883 apache#10925.
zhannngchen pushed a commit that referenced this pull request Apr 3, 2023
The offset of _nullmap and _value are inconsistent in OlapDataConvertor, so the obtained null flag is incorrect when calling get_ data_ at function. When the key column or sequence column has null values, the encoding of the short key index or primary key index may be wrong.
This was introduced by #10883 #10925.
zhannngchen pushed a commit to zhannngchen/incubator-doris that referenced this pull request Apr 3, 2023
The offset of _nullmap and _value are inconsistent in OlapDataConvertor, so the obtained null flag is incorrect when calling get_ data_ at function. When the key column or sequence column has null values, the encoding of the short key index or primary key index may be wrong.
This was introduced by apache#10883 apache#10925.
gnehil pushed a commit to gnehil/doris that referenced this pull request Apr 21, 2023
The offset of _nullmap and _value are inconsistent in OlapDataConvertor, so the obtained null flag is incorrect when calling get_ data_ at function. When the key column or sequence column has null values, the encoding of the short key index or primary key index may be wrong.
This was introduced by apache#10883 apache#10925.
mongo360 pushed a commit to mongo360/doris that referenced this pull request Jul 12, 2023
The offset of _nullmap and _value are inconsistent in OlapDataConvertor, so the obtained null flag is incorrect when calling get_ data_ at function. When the key column or sequence column has null values, the encoding of the short key index or primary key index may be wrong.
This was introduced by apache#10883 apache#10925.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. area/vectorization reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants