Skip to content

[bug](Nereids) get error row_desc() when have projection#12232

Merged
morrySnow merged 2 commits intoapache:masterfrom
HappenLee:neredis
Sep 1, 2022
Merged

[bug](Nereids) get error row_desc() when have projection#12232
morrySnow merged 2 commits intoapache:masterfrom
HappenLee:neredis

Conversation

@HappenLee
Copy link
Copy Markdown
Contributor

Proposed changes

Issue Number: close #xxx

Problem summary

Describe your 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 (If Yes, please explain WHY)
    • 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...

morrySnow
morrySnow previously approved these changes Aug 31, 2022
@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Aug 31, 2022
@github-actions
Copy link
Copy Markdown
Contributor

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

@github-actions
Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@github-actions github-actions bot removed the approved Indicates a PR has been approved by one committer. label Aug 31, 2022
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 1, 2022

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

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Sep 1, 2022
@morrySnow morrySnow merged commit 8c8078a into apache:master Sep 1, 2022
Yukang-Lian pushed a commit to Yukang-Lian/doris that referenced this pull request Sep 2, 2022
…ExecNode (apache#12232)

When ExecNode's projections is not empty, it use output row descriptor to initialize the block before doing projection. But we should use original row descriptor. This PR fix it.
Yukang-Lian pushed a commit to Yukang-Lian/doris that referenced this pull request Sep 2, 2022
…ExecNode (apache#12232)

When ExecNode's projections is not empty, it use output row descriptor to initialize the block before doing projection. But we should use original row descriptor. This PR fix it.
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.

2 participants